As a general rule I just lurk on this list. But, here are my 2 cents. . .
The suggestion of lowercasing package and module names is in alignment with the PEP 8, "Style Guide for Python Code". http://www.python.org/dev/peps/pep-0008/ According to the style guide, Package & Module names should be short and all-lowercase. Underscores can be used but should be avoided. Class names are to use CapWords. (about 2/3 of the way down in the document) Sorry, I have no input on the other questions. aaron -----Original Message----- From: Dabo-dev [mailto:[email protected]] On Behalf Of Werner F. Bruhin Sent: Monday, November 04, 2013 3:40 AM To: Dabo developers' list Subject: Re: [dabo-dev] Python 2 and 3 handling Hi all, On 03/11/2013 08:44, Werner F. Bruhin wrote: > > 1. module files are named the same as the corresponding class, I got a > bit further by renaming the files to be lower case and keep the class > as is. This is against Pep 8 and I also came across a SO post the > other day which pointed to this as an issue - which I obviously can't > find today. I found some time to have a go at this. I took dDialog.py and lower cased it to ddialog.py and then kept hacking away and lower cased everything where I got import errors on. 'ddialog.py' is now running again using absolute and/or relative imports with Python 2.7 and Phoenix, but it is not yet running with Python 3.3 (see below). Before I spend more time on this, i.e. change all modules to lower case and and and.... I like to here from you. - should all modules be lower cased? - how to handle e.g. "dKeys" and "dIcons"? I suggest to import as "import dkeys" and use as 'dkeys.whatever', to be consistent. - any ideas on the other points I made in the previous post Werner Python 3, needs more work, to start with: - I get some syntax errors in wx.lib.calendar - if above is not imported into dabo then I get this: ui.loadUI failed: currentframe() takes 0 positional arguments but 1 was given Traceback (most recent call last): File "c:\dev\dabo\dabo\ui\__init__.py", line 61, in loadUI exec("from %s import *" % mods[typ], globals()) File "<string>", line 1, in <module> File "c:\dev\dabo\dabo\ui\uiwx\__init__.py", line 97, in <module> from .dpemmixin import dPemMixin File "c:\dev\dabo\dabo\ui\uiwx\dpemmixin.py", line 21, in <module> class dPemMixin(dPemMixinBase): File "c:\dev\dabo\dabo\ui\uiwx\dpemmixin.py", line 3055, in dPemMixin DynamicBackColor = makeDynamicProperty(BackColor) File "c:\dev\dabo\dabo\ui\__init__.py", line 100, in makeDynamicProperty frame = inspect.currentframe(1) TypeError: currentframe() takes 0 positional arguments but 1 was given [excessive quoting removed by server] _______________________________________________ Post Messages to: [email protected] Subscription Maintenance: http://mail.leafe.com/mailman/listinfo/dabo-dev Searchable Archives: http://leafe.com/archives/search/dabo-dev This message: http://leafe.com/archives/byMID/[email protected]
