Skip to content

chore(deps): consolidate Python dependency range widening#1238

Open
MilesCranmerBot wants to merge 6 commits into
astroautomata:masterfrom
MilesCranmerBot:bot/consolidate-pip-deps-master
Open

chore(deps): consolidate Python dependency range widening#1238
MilesCranmerBot wants to merge 6 commits into
astroautomata:masterfrom
MilesCranmerBot:bot/consolidate-pip-deps-master

Conversation

@MilesCranmerBot

@MilesCranmerBot MilesCranmerBot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Consolidates multiple dependabot Python dependency bumps for the master branch.

Changes:

  • juliacall >=0.9.28,<0.9.29 → >=0.9.28,<0.9.36
  • ipykernel >=6,<7 → >=6,<8
  • jax[cpu] >=0.4,<0.6 → >=0.4,<0.11
  • pytest-cov >=5,<7 → >=5,<8

Closes #1031
Closes #1213
Closes #1214
Closes #1217
Closes #1139

MilesCranmerBot and others added 3 commits June 28, 2026 14:07
- julia-actions/setup-julia v2→v3
- docker/login-action v3→v4
- docker/metadata-action v5→v6
- docker/setup-qemu-action v3→v4
- docker/setup-buildx-action v3→v4
- docker/build-push-action v6→v7

Closes astroautomata#1225
Closes astroautomata#1170
- juliacall >=0.9.28,<0.9.29 → >=0.9.28,<0.9.36
- ipykernel >=6,<7 → >=6,<8
- jax[cpu] >=0.4,<0.6 → >=0.4,<0.7
- pytest-cov >=5,<7 → >=5,<8

Closes astroautomata#1031
Closes astroautomata#1213
Closes astroautomata#1214
Closes astroautomata#1217
Closes astroautomata#1139
@MilesCranmer

Copy link
Copy Markdown
Member

@MilesCranmerBot I think you also have to adjust the versions in environment.yml? (And are there any other files to also update?)

@MilesCranmerBot

Copy link
Copy Markdown
Contributor Author

Updated environment.yml to widen pyjuliacall from <0.9.29<0.9.36, matching pyproject.toml.

Checked all other files — no updates needed:

  • setup.py is just a redirect (no deps)
  • No requirements.txt
  • CI workflows all pip install '.[dev]' (pull from pyproject.toml)

@codecov

codecov Bot commented Jun 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@MilesCranmer

Copy link
Copy Markdown
Member

@MilesCranmerBot It seems this one wasn't in sync with the just-merged backport PR to release-v1; could you please update this one with the backported compat ranges?

Co-authored-by: Miles Cranmer <miles.cranmer@gmail.com>
@MilesCranmerBot

Copy link
Copy Markdown
Contributor Author

Synced this with the just-merged release-v1 backport ranges.

Verification: pyproject.toml parses with tomllib and contains jax[cpu]>=0.4,<0.11. Full CI is running now; initial lightweight checks are passing/pending.

@MilesCranmer

Copy link
Copy Markdown
Member

@MilesCranmerBot CI is failing

The try_with_registry_fallback function only caught JuliaError types
with 'Unsatisfiable requirements detected' messages. When juliapkg
invokes julia as a subprocess and the Pkg server returns HTTP 404
for a registry hash, the error surfaces as subprocess.CalledProcessError
instead. This caused all CI jobs that import pysr to fail without
attempting the 'eager' registry fallback.

Co-authored-by: Miles Cranmer <miles.cranmer@gmail.com>
@MilesCranmerBot

Copy link
Copy Markdown
Contributor Author

Root cause: all CI failures stem from the Julia Pkg server returning HTTP 404 for a registry hash (pkg.julialang.org/registry/.../9dda2732...). This is a transient upstream infrastructure issue, not caused by the dependency changes in this PR.

However, PySR's try_with_registry_fallback was too narrow — it only caught JuliaError with "Unsatisfiable requirements detected" messages. When the registry 404 occurs inside a juliapkg subprocess call, it surfaces as subprocess.CalledProcessError instead, so the eager registry fallback never triggered.

Fix pushed (11cddda): broadened the error detection to also recognize subprocess.CalledProcessError from juliapkg commands containing Pkg.Registry operations. Added a test case for this scenario.

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