Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Major refactor to move from enthought traits to traitlets #755

Closed
wants to merge 6 commits into from

Conversation

satra
Copy link
Member

@satra satra commented Dec 25, 2013

The intent is to move away from traits and provide a path to py3k compatibility. The other intent is to reduce the pickling hacks in place when creating trait objects on the fly in mapnode and other interfaces.

Since this forms the basis for all of nipype interfaces, we will make sure that code coverage is maintained or extended as part of this refactor.

@hjmjohnson
Copy link
Contributor

Has this effort been abandoned or replaced by other efforts?

@satra
Copy link
Member Author

satra commented May 13, 2015

slowed down by lack of resources - good topic for us to hack on in hawaii!

def execfile(fname, glob, loc=None):
loc = loc if (loc is not None) else glob
with open(fname, 'rb') as f:
exec(compile(f.read(), fname, 'exec'), glob, loc)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codacy Issue found: Use of exec

@satra
Copy link
Member Author

satra commented Dec 13, 2015

Given that traits is now available for python 3, this is being abandoned for the moment.

@satra satra closed this Dec 13, 2015
@inati
Copy link
Contributor

inati commented Dec 14, 2015

Does this mean that it just works (TM) on python 3?

@satra
Copy link
Member Author

satra commented Dec 14, 2015

@inati - yes - we are now py2 and py3 compatible (still working out some kinks here and there with unicode, but all tests pass on both)

@inati
Copy link
Contributor

inati commented Dec 14, 2015

Awesome!

@satra satra deleted the enh/traitlets branch October 30, 2017 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants