updated all __all__ lists in each module#401
updated all __all__ lists in each module#401mihaic merged 7 commits intobrainiak:masterfrom lcnature:fix_all__
Conversation
brainiak/hyperparamopt/hpo.py
Outdated
There was a problem hiding this comment.
I think only fmin should be listed here. I made issue #402 about renaming the others, but we could already list only what is necessary in __all__. What do you think?
There was a problem hiding this comment.
I am fine with that. I did not read into detail of what other commands do.
| "ConditionSpec", | ||
| "SingleConditionSpec", | ||
| "mask_image", | ||
| "MaskedMultiSubjectData", |
There was a problem hiding this comment.
I was trying to ordered __all__ alphabetically.
brainiak/image.py
Outdated
There was a problem hiding this comment.
Let's add a comma after the last element as well in multi-line lists, to minimize the changes made later on. See PEP 8:
https://www.python.org/dev/peps/pep-0008/#when-to-use-trailing-commas
brainiak/isc.py
Outdated
There was a problem hiding this comment.
@snastase, are all these supposed to be used outside BrainIAK?
There was a problem hiding this comment.
Okay, I fixed up all and prepended underscores in the appropriate spots for isc.py
brainiak/utils/utils.py
Outdated
There was a problem hiding this comment.
Could you please insert the new ones alphabetically?
some functions are removed from __all__ in hpo.py
|
See PR #408 for a fix for the current build failure. |
This PR is to fix the issue of #253