Skip to content

Add workflow to build pyodide package#3128

Open
airen1986 wants to merge 5 commits into
ERGO-Code:latestfrom
airen1986:master
Open

Add workflow to build pyodide package#3128
airen1986 wants to merge 5 commits into
ERGO-Code:latestfrom
airen1986:master

Conversation

@airen1986

Copy link
Copy Markdown

With the acceptance of PEP 783, package maintainers can now build and publish Pyodide-compatible wheels (pyemscripten platform) directly to PyPI. Submiting this PR to build python package in HiGHS repository, will update the code in build-wheels-push.yml, so pyodide packages can be uploaded to PyPI

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.16%. Comparing base (dfd5cbf) to head (51b6ab2).
⚠️ Report is 44 commits behind head on latest.

Additional details and impacted files
@@           Coverage Diff           @@
##           latest    #3128   +/-   ##
=======================================
  Coverage   73.15%   73.16%           
=======================================
  Files         430      430           
  Lines      103937   103967   +30     
  Branches    16717    16718    +1     
=======================================
+ Hits        76039    76069   +30     
  Misses      27622    27622           
  Partials      276      276           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jajhall jajhall requested a review from galabovaa July 2, 2026 08:07
@jajhall jajhall self-assigned this Jul 2, 2026
@galabovaa galabovaa requested a review from mathgeekcoder July 2, 2026 12:02
@galabovaa

Copy link
Copy Markdown
Contributor

LGTM if @mathgeekcoder is happy as well

@galabovaa

Copy link
Copy Markdown
Contributor

Should be unrelated but we get a first sanitizer failure here @Opt-Mucca would you look at it please?

@Opt-Mucca

Copy link
Copy Markdown
Collaborator

@galabovaa Can do!

@airen1986

Copy link
Copy Markdown
Author

@Opt-Mucca Also note that I have excluded test_user_interrupts and test_with_context_manager tests from pyodide build as those depends on threads/multithreading, which pyodide doesn't support and those tests are falling

@Opt-Mucca

Copy link
Copy Markdown
Collaborator

@airen1986 Just to clarify: That comment is unrelated to the sanitizer build being run in this pipeline?

@airen1986

Copy link
Copy Markdown
Author

@airen1986 Just to clarify: That comment is unrelated to the sanitizer build being run in this pipeline?

yes, it is unrelated, just something to note for the team

@Opt-Mucca

Copy link
Copy Markdown
Collaborator

@galabovaa Locally I'm not getting any errors for tsan and 250 runs_per_test. I can't seem to get the log of the single run that failed in pipeline too.

@mathgeekcoder

Copy link
Copy Markdown
Collaborator

LGTM if @mathgeekcoder is happy as well

I'm a little hesitant to approve just yet. I don't know what the ramifications are with officially(?) supporting WASM.

As @airen1986 mentioned, threading isn't really supported. I think we might need to make some minor changes to highspy. I don't know if changes are required for the C++ threading etc. too.

@mathgeekcoder

Copy link
Copy Markdown
Collaborator

I should've mentioned that I think this sounds really cool though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants