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

Add WASM wheels build #1653

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add WASM wheels build #1653

wants to merge 2 commits into from

Conversation

gbolmier
Copy link
Member

@gbolmier gbolmier commented Nov 28, 2024

Let's first test the build of the WASM wheels in an independant workflow so that it is easier to iterate and make sure it works before adding the logic to the pypi workflow.

See #1649 thread for more context.

Current blocker is a conflict between River pandas<3.0.0,>=2.2.3 requirement and Pyodide pandas version (2.2.0) (see run error).

Let's first test the build of the WASM wheels in an independant
workflow so that it is easier to iterate and make sure it works
before adding the logic to the pypi workflow.
@gbolmier gbolmier force-pushed the build-wasm-wheels branch 2 times, most recently from e780d5f to e0fb7b5 Compare November 28, 2024 05:09
@MaxHalford
Copy link
Member

Great stuff @gbolmier!

For the pandas version, I guess it's fine to loosen it on our end.

@gbolmier
Copy link
Member Author

For the pandas version, I guess it's fine to loosen it on our end.

I would prefer too, if possible. @agriyakhetarpal do you remember why you bumped pandas from ^2.1 to ^2.2.3 in #1632?

I just realized that Pyodide is not supporting NumPy >=2.0 yet (see supported list). That's why their supported pandas version is 2.2.0 (which is not supporting NumPy >=2.0 (see reqs).

@agriyakhetarpal does it mean that we need to wait for Pyodide to support newer pandas and NumPy versions in order to test River's WASM build?

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