Feature/textual gui #79
Conversation
htwangtw
left a comment
htwangtw
left a comment
There was a problem hiding this comment.
Amazing, some small tweaks
requirements.txt
Outdated
There was a problem hiding this comment.
Can yoy integrate these into the pyproject.toml? Python is gradually moving to have all the configurations integrated in one file
Co-authored-by: Hao-Ting Wang <htwangtw@gmail.com>
|
merged requested tweaks and moved dependancies to .toml instead of requirements.txt. GUI stack can be downloaded with pip install .[textual] |
htwangtw
left a comment
htwangtw
left a comment
There was a problem hiding this comment.
Okay, the test is still failing - we need to have a condition to check if the relevant package is installed before the imports. We can have a look at this:
Nilearn has similar checks for matplotlib:
https://github.com/nilearn/nilearn/blob/73f30611318b3586ae155b086d075f8a80abfabd/nilearn/_utils/helpers.py#L294-L316
And it is used in their conftest to determine whether the test requires a given package.
We can wait until the new year for this one
This PR corresponds to files for the new textual UI.
To use the new UI, please run pip install -r requirements.txt
then pip install -e .
then wonkyconn --textual