Skip to content

Upgrade notebooks to v0.3.0 of the online lab #36

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

Merged
merged 30 commits into from
Apr 25, 2025
Merged

Upgrade notebooks to v0.3.0 of the online lab #36

merged 30 commits into from
Apr 25, 2025

Conversation

juntyr
Copy link
Member

@juntyr juntyr commented Apr 3, 2025

Adapts the notebooks in preparation for v0.3 of the online lab, which updates to Python 3.13, updates all packages, and updates the compression API to use public packages (instead of the in-dev fcbench).

@juntyr juntyr requested a review from SF-N April 3, 2025 09:57
@SF-N
Copy link
Contributor

SF-N commented Apr 3, 2025

I started taking a look and got an error when running the 01-intro.ipynb notebook, when trying to load and run it again, e.g. by calling https://lab.climet.eu/latest/github/climet-eu/compression-lab-notebooks/update-v0-3, only the README.md and the requirements.txt are loading in JupyterLite and an empty compression-lab-notebooks folder.
I had about 5 successful attempts of accessing the online lab until I faced this issue.
I thought the limit of accesses would be much higher than about 5 within half an hour. Does it currently work for you, and do you have any suggestions?

@juntyr
Copy link
Member Author

juntyr commented Apr 3, 2025

You need to use the lab main branch (latest is still v0.2.1): https://lab.climet.eu/main/github/climet-eu/compression-lab-notebooks/update-v0-3

I think the rate limit is ~60 files per hour, so given that our repo now has more than 10, this makes sense.

@juntyr
Copy link
Member Author

juntyr commented Apr 3, 2025

I've opened jupyterlite/litegitpuller#27

@SF-N
Copy link
Contributor

SF-N commented Apr 3, 2025

You need to use the lab main branch (latest is still v0.2.1): https://lab.climet.eu/main/github/climet-eu/compression-lab-notebooks/update-v0-3

This https://lab.climet.eu/latest/github/climet-eu/compression-lab-notebooks/update-v0-3 is what I get from the docs when entering Repository owner: climet-eu, Repository name: compression-lab-notebooks and Git ref: update-v0-3

Copy link
Contributor

@SF-N SF-N left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With your link the 01-intro.ipynb notebook works fine.
You can go ahead and merge from my point of view.

@juntyr
Copy link
Member Author

juntyr commented Apr 3, 2025

You need to use the lab main branch (latest is still v0.2.1): https://lab.climet.eu/main/github/climet-eu/compression-lab-notebooks/update-v0-3

This https://lab.climet.eu/latest/github/climet-eu/compression-lab-notebooks/update-v0-3 is what I get from the docs when entering Repository owner: climet-eu, Repository name: compression-lab-notebooks and Git ref: update-v0-3

It feels like maybe the version names are confusing? Currently, latest refers to the latest published version, while main refers to the main branch (in-development) version. Readthedocs calls the latest published version stable, would that be clearer? We could also give an alias to main (since it's just the branch name), e.g. dev or nightly (which is often used to imply that thinks will break on this version) even latest? I'm open to any change that makes the names less confusing.

Also, do you think the main option should be added to the docs GUI? I didn't for now since I didn't want to drive users to it, but perhaps it might be useful if we choose an alias like nightly that communicates that it's a version that's not meant to work all the time

@SF-N
Copy link
Contributor

SF-N commented Apr 4, 2025

It feels like maybe the version names are confusing? Currently, latest refers to the latest published version, while main refers to the main branch (in-development) version. Readthedocs calls the latest published version stable, would that be clearer? We could also give an alias to main (since it's just the branch name), e.g. dev or nightly (which is often used to imply that thinks will break on this version) even latest? I'm open to any change that makes the names less confusing.

Also, do you think the main option should be added to the docs GUI? I didn't for now since I didn't want to drive users to it, but perhaps it might be useful if we choose an alias like nightly that communicates that it's a version that's not meant to work all the time

Thanks for the explanation, now I understand.
I would leave latest as it is now and add main with an alias (I would prefer dev over nightly).

@juntyr juntyr changed the title Prepare update to the online lab v0.3 Upgrade notebooks to v0.3.0 of the online lab Apr 25, 2025
@juntyr juntyr merged commit d297ee9 into main Apr 25, 2025
@juntyr juntyr deleted the update-v0-3 branch April 25, 2025 14:10
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

Successfully merging this pull request may close these issues.

2 participants