-
Notifications
You must be signed in to change notification settings - Fork 0
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
get_projects is failing with "['project_code'] not in index" #16
Comments
We have to update the images used by Jupyterhub, so that they references aquamonitor-python 0.3.3. |
I have temporarily downloaded older version on jupyter hub with !pip install --user --force-reinstall git+https://github.com/NIVANorge/Aquamonitor-Python.git@6cf4abf since the updated versions cause incompatibility. There are still some messages, but the functions seem to work, I will check more |
no, I was wrong, it did not work, because the next commit (that has the error in the func) has the same version, I guess this is the resason why git still installs it. I can make a branch, downgrade libraries and install from the branch, is it ok for you? |
I don't see why you should downgrade. The 0.3.3 version works. |
becuase current versions of libraries joblib>=1.4 |
Is it necessary for AM-Python to have such strict dependencies, @roarbra ? E.g. here I see we now require Having strict dependency pinning is fine if necessary, but otherwise it makes dependency resolution on the Hub harder, as outlined here. @lisapro On your Hub, try
I've just tested on my Hub and it successfully installs |
@JamesSample the --no-deps option fixed it, yes! |
No, not at all. I'm just pushing these versions to be up to date, and avoid warnings about vulnerabilities. |
I guess vulnerabilities are a good reason to keep updating the pinning. Maybe we can try to find a balance somewhere in the middle? You can also keep doing exactly as you are, because
should work, unless one day AM-Python really does rely on new features only available in the updated |
Importmal app is using Aquamonitor-Python for getting the data https://github.com/NIVANorge/importmal_app/tree/master
I use it though Jupyter Hub
Now it fails on get_projects function, I see that there were recent changes in it. Could you please take a look?
The text was updated successfully, but these errors were encountered: