Skip to content
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

Release button dose not work normally (Jupyterhub with dockerspawner ) #169

Open
guyq1997 opened this issue Apr 18, 2024 · 4 comments
Open

Comments

@guyq1997
Copy link

Hello Guys,

i am using ngshare in Jupyterhub with dockerspawner.

Everything goes well if i only use nbgrader.

After i used ngshare, and specify the exchange directory to ngshare API (/services/ngshare). The Release Button does not work.

The Log reported 'Successfully released':
image

But actually there is nothing released, and the Status has not changed at all:
image

But i can see the Assignments are successfully stored in /srv/ngshare of jupyterhub container.

@jueren
Copy link

jueren commented Aug 28, 2024

Hello @guyq1997,

we ran into a similar problem and after days(!) of investigation we found out the following:

  • When ngshare is installed with pip the "regular" way (i.e. pip install ngshare) and using pip list it shows SQLAlchemy in Version 2.0.30 (however its clearly bounded to a version < 2.0 as you can see here: https://github.com/LibreTexts/ngshare/blob/master/setup.py)
  • When ngshare is installed with pip -e /path/to/freshly-checked-out-ngshare-source-code/here pip list shows SQLAlchemy in a compatible version (1.4.53 in our case) - and everything seems to work just fine.

Have a look here:

Screenshot 2024-08-27 at 18 02 00

Whether it is a pip issue or bug in setuptools, we do not know yet. I hope, the folks at LibreTexts can have a look at it.

I hope, that this information may help you figure out, why you are also unable to do write operations to the sqlite-db (fileupload was working on our side as well)

Cheers

@ykazakov
Copy link

The limit < 2.0 of SQLAlchemy was introduced after the release of v0.6.0.

But it is easy to set this limit manually:

pip install ngshare "SQLAlchemy<2.0"

@jueren
Copy link

jueren commented Sep 27, 2024

Hi Ykazakov,

thank you for clarifying. that explains the different behaviour.

its very irritating (to say the least) to have an officially released version 0.6.0 that actually does not work. no indication, hints, warnings, logs (while executing the code or in the documentation) ... failing quietly is considered very bad practice.
please release the fix of this particular severe bug as a new version as soon as possible.

and no: one does NOT want to install specific version of a subpackage when adding a dependency. we want to add a sound and consistent package installed in a specific version.

Cheers

@moorepants
Copy link
Member

@jueren Your last message is not appreciated and nor does it invoke any cheer. What is irritating are comments like yours made to people that provide you software for free. Maybe you are unfamiliar with how open software is developed. Our OSS license precisely specifies this:

THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.

If you don't like how it works, then you are welcome to contribute fixes or to fork it and use it as you like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants