Skip to content

chore(deps): bump itis-sumo 0.1.0a10 -> 0.1.0a11 (additive report facade; zero pixel drift) - #694

Closed
JavierGOrdonnez wants to merge 2 commits into
developfrom
jgo/sumo-pin-a11
Closed

JavierGOrdonnez wants to merge 2 commits into
developfrom
jgo/sumo-pin-a11

Conversation

@JavierGOrdonnez

@JavierGOrdonnez JavierGOrdonnez commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

What

Bump itis-sumo==0.1.0a10 → 0.1.0a11 (TestPyPI). a11 (itis-sumo#54) promotes incubator tooling and lands the curated itis_sumo.report facade for notebook consumers — additive to the itis_sumo.api surface this repo consumes. Follow-on tracked on the portage ledger after the backend portage (#688) landed.

Syncs: pyproject.toml, uv.lock, and the flaskapi/SPEC.md §C compute-pin line. The §B B32gs row deliberately keeps a10 (historical record — that bug's fix shipped with a10).

Verification (all local, then CI)

  • flaskapi: 582 passed / 3 skipped @ 93 %; analytical Tier-3 (real Dakota through the bumped engine): 2/2
  • e2e inside pinned mcr.microsoft.com/playwright:v1.63.0-noble: 20/20, zero pixel drift — no baseline regen in this PR (none needed; §C keeps baselines fixed across refactors)
  • uv lock also corrects a stale root-version stamp in the lockfile (1.6.3 → 1.6.3.dev2)

Addendum: CI-hardening commits (the bump surfaced a live trap)

First push was green except verify-image-build: my uv lock run mirrored the auto-version bot's PEP440 stamp (pyproject = 1.6.3.dev2, stamped post-#688) into uv.lock, and the image build's oSPARC MetadataConfig validates the lock-derived version as semver. The lock's released 1.6.3 on develop is load-bearing.

16a95349 carries the minimal repair:

  1. restores uv.lock project version to 1.6.3 (develop parity),
  2. switches the ty hook from uv run to uv run --frozen — without this, every future flaskapi-touching PR would face the same wall: the hook's auto-sync rewrites the lock (breaking commit/prek) or the rewritten stamp breaks verify-image-build. This alone unblocks the queue; a systemic fix for the stamp chain is filed separately.

@codecov

codecov Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.50%. Comparing base (3d16d69) to head (def6c13).
⚠️ Report is 15 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #694       +/-   ##
============================================
+ Coverage    50.63%   85.50%   +34.87%     
============================================
  Files          107      109        +2     
  Lines         5350    12978     +7628     
  Branches       605     1352      +747     
============================================
+ Hits          2709    11097     +8388     
+ Misses        2563     1770      -793     
- Partials        78      111       +33     
Flag Coverage Δ
e2e 79.13% <ø> (?)
flaskapi 93.49% <ø> (+<0.01%) ⬆️
node 64.55% <ø> (+39.77%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The dependency declaration, lockfile, and specification are consistent, with no unresolved compatibility issues found.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the Flask API’s itis-sumo dependency to additive release 0.1.0a11.

Changes:

  • Bumps and locks itis-sumo to 0.1.0a11.
  • Synchronizes the lockfile’s root package version.
  • Updates the documented compute pin.
File Description
flaskapi/​pyproject.toml Updates the dependency pin.
flaskapi/​uv.lock Locks the new artifacts and synchronizes package metadata.
flaskapi/​SPEC.md Documents the updated compute version.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

JavierGOrdonnez pushed a commit that referenced this pull request Oct 6, 2026
…frozen

Two sides of one trap the auto-version bot set today:
- the bot stamps pyproject with PEP440 dev versions (1.6.3.dev2); a
  plain 'uv run' (ty hook) auto-syncs and mirrors that stamp into
  uv.lock -- whose project version feeds the image build's oSPARC
  MetadataConfig, which validates SEMVER (verify-image-build failed
  on #694's first push for exactly this). This commit restores the
  lock to 1.6.3 (as on develop) and switches the hook to
  'uv run --frozen' (installs from lock as-is, mirroring the CI
  'uv sync --frozen' convention), so no future flaskapi-touching PR
  can be forced to propagate the dev stamp into the lock.
@JavierGOrdonnez

Copy link
Copy Markdown
Collaborator Author

verify-image-build retest after 16a95349 failed with the same ValidationError: version [string_pattern_mismatch, input_value='1.6.3.dev2'] — and it turns out this PR cannot fix it in-PR: the version feeding MetadataConfig is .osparc/flaskapi/metadata.yml, which the auto-tag bot's dev stamp fans out via .bumpversion.cfg (all 8 metadata.yml files carry 1.6.3.dev2 on develop since 195632ec, an ancestor of the current tip). The lock restore stays valuable as the --frozen guard, but it isn't the metadata source.

Consequence: every PR branched from today's develop fails verify-image-build regardless of content. Root-cause + fix shipped as #697 (semver-only fanout entries + one-time metadata repair). Once #697 merges, this PR only needs a re-run of the failed check — no rebase.

Javier Garcia Ordonez added 2 commits October 7, 2026 09:40
a11 (itis-sumo#54) promotes the incubator tooling + adds the curated
itis_sumo.report facade for notebook consumers - additive to the
itis_sumo.api surface flaskapi consumes.

- flaskapi 582 pass/3 skip @93%; analytical (real Dakota) 2/2
- e2e in pinned playwright:v1.63.0-noble: 20/20, ZERO pixel drift
  (no regen needed; baselines untouched by design)
- SPEC: §C compute pin line synced; §B B32gs history row left at
  a10 (historical record: that fix landed WITH a10)
- uv.lock also syncs the stale self-version stamp (1.6.3 -> 1.6.3.dev2)
…frozen

Two sides of one trap the auto-version bot set today:
- the bot stamps pyproject with PEP440 dev versions (1.6.3.dev2); a
  plain 'uv run' (ty hook) auto-syncs and mirrors that stamp into
  uv.lock -- whose project version feeds the image build's oSPARC
  MetadataConfig, which validates SEMVER (verify-image-build failed
  on #694's first push for exactly this). This commit restores the
  lock to 1.6.3 (as on develop) and switches the hook to
  'uv run --frozen' (installs from lock as-is, mirroring the CI
  'uv sync --frozen' convention), so no future flaskapi-touching PR
  can be forced to propagate the dev stamp into the lock.
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