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

_______________________________________________
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]

Reply via email to