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

[0.1.x] pyodide 0.24.0, widget shim patches #62

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

bollwyvl
Copy link
Contributor

@bollwyvl bollwyvl commented Sep 14, 2023

References

Changes

  • update pyodide version references
  • rename repodata.json to pyodide-lock.json
  • update for micropip 0.4.0
  • update widget shims vs ipywidgets 8.1.1

Notes

@bollwyvl bollwyvl changed the base branch from main to 0.1.x September 14, 2023 14:47
@github-actions
Copy link
Contributor

lite-badge 👈 Try it on ReadTheDocs

@bollwyvl bollwyvl marked this pull request as ready for review September 14, 2023 14:57
@bollwyvl
Copy link
Contributor Author

Looking fine on RTD.

@bollwyvl bollwyvl changed the title pyodide 0.24.0, widget shim patches [0.1.x] pyodide 0.24.0, widget shim patches Sep 14, 2023
@bollwyvl bollwyvl requested a review from jtpio September 14, 2023 15:04
@jtpio
Copy link
Member

jtpio commented Sep 15, 2023

  • could target main, and then re-do a bunch of stuff

We can ask Meeseeks to backport to main. Although #58 also seems to be updating widgetsnbextension so potential conflics.

@michaelweinold
Copy link

@jtpio, since you asked back in #61:

peewee (and sqlite3 in general) work fine now 🎉

from peewee import *

# Define a database object
db = SqliteDatabase('/tmp/my_database.db')

# Define a model class
class Person(Model):
    name = CharField()
    age = IntegerField()

    class Meta:
        database = db

# Create the table in the database
db.connect()
db.create_tables([Person])

This of course with the limitations already discussed in:

Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit e39526f into jupyterlite:0.1.x Sep 15, 2023
10 checks passed
@jtpio
Copy link
Member

jtpio commented Sep 15, 2023

@meeseeksdev please backport to main

meeseeksmachine pushed a commit to meeseeksmachine/pyodide-kernel that referenced this pull request Sep 15, 2023
jtpio pushed a commit that referenced this pull request Sep 15, 2023
Co-authored-by: Nicholas Bollweg <nick.bollweg@gmail.com>
@bollwyvl
Copy link
Contributor Author

sqlite3 in general work fine now

my heart jumped a little until the disclaimer...

of course with the limitations already discussed

😿

@michaelweinold
Copy link

michaelweinold commented Sep 15, 2023

@bollwyvl, if you're interested in moving the sql filesystem issues forward, please get in touch with me at (email). We might have some financial resources available to hire someone on UpWork to work on this specific issue.

@bollwyvl
Copy link
Contributor Author

For my purposes, public comms are preferred, but certainly we could set up a call on the Jupyter zoom or something if there are folk that want to talk about it... i'm in US/Central, but can be pretty flexible on times. Usually right before (or after, but now getting late for GMT+) the block of Notebook/Lab calls on Wednesdays.

The challenge is the potential developer pool for ServiceWorker/DriveFS support is... very small. It's been a fair piece since i tracked down a few of the remaining issues (probably before the split to a dedicated repo). My day job test developer instinct would be to fully instrument down to the (virtual) syscall level on both real CPython instance, one the MEMFS that does work (e.g. what you get in Firefox private browsing), and one against the ServiceWorker-backed one... but I don't have an effective test bench for doing that.

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

Successfully merging this pull request may close these issues.

Option "Using a custom Pyodide distribution" (specified in jupyter-lite.json) not working
3 participants