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

Updated Jupyterlab #451

Merged
merged 4 commits into from
Mar 8, 2023
Merged

Updated Jupyterlab #451

merged 4 commits into from
Mar 8, 2023

Conversation

GTrunSec
Copy link
Collaborator

@GTrunSec GTrunSec commented Mar 5, 2023

fixes: #440

@netlify
Copy link

netlify bot commented Mar 5, 2023

Deploy Preview for tweag-jupyterwith ready!

Name Link
🔨 Latest commit 5948d4e
🔍 Latest deploy log https://app.netlify.com/sites/tweag-jupyterwith/deploys/640668544570e700080cd4f5
😎 Deploy Preview https://deploy-preview-451--tweag-jupyterwith.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@GTrunSec GTrunSec marked this pull request as ready for review March 5, 2023 23:04
@garbas
Copy link
Contributor

garbas commented Mar 6, 2023

@GTrunSec Can you only include f13ff5d commit and not the rest?

@GTrunSec
Copy link
Collaborator Author

GTrunSec commented Mar 6, 2023

@garbas sure.

Copy link
Contributor

@garbas garbas left a comment

Choose a reason for hiding this comment

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

Looks good to me, except the preferWheels, change.

I've looked at the release notes since jupyterlab 3.5.0 and found this two PRs might be cause some problems since they touch some cirectories stuctures:

At the same time I found an interesting PR we should be keeping an eye on since it will allow us to have less hacky per environment config: jupyterlab/jupyterlab#12916

lib/jupyter.nix Outdated
@@ -23,7 +23,7 @@
python ? pkgs.python3,
editablePackageSources ? {},
extraPackages ? (ps: []),
preferWheels ? false,
preferWheels ? true,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
preferWheels ? true,
preferWheels ? false,

Why is this needed? I'd prefer to build without wheels since that can cause some weird problems by my experience. I understand this options might be interesting for final users since it would speed things up, but for core stuff here we can be on the side of causion and build things from source.

Copy link
Collaborator Author

@GTrunSec GTrunSec Mar 6, 2023

Choose a reason for hiding this comment

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

cause some weird problems

So hard to resolve some dependencies. We can override some packages for building without preferWheels, such as jupyterlab, jupyter -server.

@djacu djacu merged commit 0c03a8e into tweag:main Mar 8, 2023
@GTrunSec GTrunSec deleted the jupyterlab-update branch March 26, 2023 22:45
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.

JupyterLab IHaskell cell execution number
3 participants