Skip to content

Conversation

cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Jan 23, 2025

Adds a new build matrix element to build and test the latest release of Pyodide with DuckDB.

@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 23, 2025

Looks like these builds are failing. I can look into that if you're not already doing that @carlopi.

@carlopi
Copy link
Contributor

carlopi commented Jan 23, 2025

The test part is currently failing, yes, the first failure is in a spark-connected test.

I have not looked up there whether the problem is somehow on our side (and can be handled). One possibility that is fine is also just skip the test (as long as the failure is in some experimental or not super relevant part / not a sign of some other weakness).

@duckdb-draftbot duckdb-draftbot marked this pull request as draft January 23, 2025 15:50
@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 23, 2025

Hm, looks like there's at least 3 problems:

  1. The 0.26.1 wheel artifact has the same name as the 0.27.* artifact
  2. There's a test failure regarding type stubs
  3. There's an interpreter crash regarding failure to convert an undefined into a BigInt

I'll close this out for now until I can figure out what's going on.

@cpcloud cpcloud closed this Jan 23, 2025
@cpcloud cpcloud deleted the add-new-pyodide-version branch January 23, 2025 16:03
carlopi added a commit to carlopi/duckdb that referenced this pull request Jan 27, 2025
carlopi added a commit to carlopi/duckdb that referenced this pull request Jan 30, 2025
carlopi added a commit to carlopi/duckdb that referenced this pull request Jan 30, 2025
Mytherin added a commit that referenced this pull request Jan 30, 2025
Some more touches to Pyodide tests / CI, plus adding @cpcloud's
#15862 that enabled Pyodide 0.27.2.
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