-
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
Update "install_requires" from aiida to aiida-core #20
Comments
Thanks a lot for these suggestions, will look into this! |
fix aiidateam/aiida-plugin-cutter#20 * limit aiida version to <1.0 * switch from aiida to aiida-core * use aiida's prerequisites for pre-commit
I can reproduce the pgtest import error The travis log shows that
It turns out that putting the same dependency with different extras is an unresolved pip issue. |
address aiidateam/aiida-plugin-cutter#20 * limit aiida version to <1.0 * switch from aiida to aiida-core * add dependencies explicitly, working around pypi bug pypa/pip#4957
fix aiidateam#20 * limit aiida version to <1.0 * switch from aiida to aiida-core * add dependencies explicitly, working around pypi bug pypa/pip#4957 * switch to aiida 0.12 as default minimum version
One of the major reasons to do this is that readthedocs builds seem to fail with aiida (https://readthedocs.org/projects/aiida-crystal17/builds/7693840/) but not with aiida-core
I'd also note that, for some reason I get an
ImportError
forpgtest
during tests, unless I specifically but it in the requirements (even though it seems to be in the aiida-core[testing] requirements?)The text was updated successfully, but these errors were encountered: