-
Notifications
You must be signed in to change notification settings - Fork 12
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
MAINT: Switch to a setup.cfg
-style of installation
#24
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. A few comments...
TF_LAYOUT = Layout( | ||
str(TF_HOME), validate=False, config='templateflow', | ||
ignore=['.git', '.datalad', '.gitannex', '.gitattributes', 'scripts']) | ||
except ImportError: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What condition should this happen under? You've got pybids ~= 0.9.2
in your install_requires
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason pybids is not available when running the update script during install - could dig more into this, but I'm not sure it's worthy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, you could add pybids to setup_requires
and pyproject.toml
. Very heavy lifts, though...
from .bids import Layout | ||
TF_LAYOUT = Layout( | ||
str(TF_HOME), validate=False, config='templateflow', | ||
ignore=['.git', '.datalad', '.gitannex', '.gitattributes', 'scripts']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These may be automatically ignored these days... I might flag them for review in a future PR, though not worth the trouble for this PR, IMO.
Co-Authored-By: Chris Markiewicz <markiewicz@stanford.edu>
No description provided.