-
Notifications
You must be signed in to change notification settings - Fork 20
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
Solid surfinpy josh #27
Conversation
setup.py
Outdated
@@ -23,7 +23,8 @@ | |||
license='MIT license', | |||
packages=['surfinpy'], | |||
zip_safe=False, | |||
install_requires=['scipy', 'numpy', 'jupyter', 'pymatgen', 'numpy', 'matplotlib', 'coverage', 'coveralls', 'seaborn'], | |||
python_requires='>=3.6', | |||
install_requires=['scipy', 'numpy', 'jupyter', 'pymatgen', 'numpy', 'matplotlib', 'seaborn','pyyaml'], |
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.
Is jupyter an actual install requirement here (see #22)? Also, you added pandas to list of dependencies in the README, which is indirectly included via seaborn, however shouldn't it be listed explicitly here?
Further the install requirements don't have version constraints, maybe worth adding them if they are in the requirements.txt
which is not available via an install from PyPI.
- Removed Pandas from readme - Removed version constraints in install - Updated SciPy constants to use value over depreciated codata.
Removed specific versions from requirements.txt. |
We aspire to deal with all pull requests in a timely manner. Please be patient.
Ideally pull requests will respect the underlying intrastructure of surfinpy and agree with PEP8, etc. However, please do not let this put novice programmers off contributing as we will do everything we can to help.