-
Notifications
You must be signed in to change notification settings - Fork 3
Python kernel should be able to import geopandas #239
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
base: main
Are you sure you want to change the base?
Conversation
Hmm, the reason this doesn't work is because |
As above – we'd need to either add geopandas to the lockfile (with a new lockfile URL, that is) or ship it with the kernel at build time
for more information, see https://pre-commit.ci
No, that wouldn't work. We'll either need to ship it here: https://jupyterlite.readthedocs.io/en/latest/howto/pyodide/wheels.html with the artifacts (which I don't recommend – it's a big binary we don't have a dedicated CDN to serve it). It is not being built from its recipe right now because it depends on |
Or, we would need to use an older Pyodide kernel with Pyodide 0.27, as we have it there: https://cdn.jsdelivr.net/pyodide/v0.27.6/full/pyodide-lock.json?from-lite-config=1. That won't be easy, as the Pyodide kernel and the Pyodide version within it are coupled, and the former pins JupyterLite. Going to an older version of JupyterLite would be undesirable for us. |
Is there a reason newer |
It was just because Yes, I think we can help upstream by fixing the |
geopandas
can be imported #225