Skip to content

Add async ICLabel browser execution - #384

Merged
neuromechanist merged 5 commits into
epic/324-pyodide-browserfrom
feature/issue-378-phase5-iclabel-web
Sep 18, 2026
Merged

neuromechanist merged 5 commits into
epic/324-pyodide-browserfrom
feature/issue-378-phase5-iclabel-web

Conversation

@neuromechanist

Copy link
Copy Markdown
Member

Summary

  • Add iclabel_async and pop_iclabel_async for Emscripten/Pyodide execution, with clear synchronous fail-fast errors.
  • Add the ONNX Runtime Web bridge, Pyodide runner integration, async console and GUI session coordination, and stale-result protection.
  • Add native-vs-browser parity tooling, CI coverage, user help, API docs, and workflow documentation.

Validation

  • Focused tests: 26 passed, 2 skipped (MATLAB/PyTorch optional).
  • Native/browser sample parity: allclose: true, shape [32, 7], max absolute difference 4.32133674621582e-06 at rtol=1e-4, atol=1e-5.
  • ./pre-commit.py --changed-from origin/develop passed.
  • Ruff, format, MJS syntax, and diff checks passed.

Fixes #378

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's extra usage balance is too low to start another review, so this review was not started.

An organization admin can add extra usage credits at claude.ai/admin-settings/usage.

Once credits are available, reopen this pull request to trigger a review.

@claude

claude Bot commented Sep 17, 2026

Copy link
Copy Markdown

Claude encountered an error after 0s —— View job


I'll analyze this and get back to you.

@neuromechanist
neuromechanist merged commit e6a9525 into epic/324-pyodide-browser Sep 18, 2026
8 checks passed
@neuromechanist
neuromechanist deleted the feature/issue-378-phase5-iclabel-web branch September 18, 2026 02:18
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.

1 participant