⬆️ dep-bump: py3.14 support#515
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Coordinax’s packaging metadata and CI matrix to explicitly support Python 3.14, along with dependency minimum bumps likely required for 3.14 compatibility.
Changes:
- Add Python 3.14 classifiers across workspace packages and include 3.14 in the CI test matrix.
- Bump minimum versions for key runtime deps (quax/quax-blocks/quaxed/unxt) and switch to
plum-dispatchwith a Python-3.14-specific minimum version. - Pin the
re-actors/alls-greenGitHub Action to a specific commit hash.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Adds Python 3.14 classifier; updates core dependency constraints (incl. conditional plum-dispatch and bumps for quax/quaxed/unxt). |
| packages/coordinax.interop.astropy/pyproject.toml | Updates plum-dispatch and unxt requirements with Python 3.14 conditional. |
| packages/coordinax.hypothesis/pyproject.toml | Adds Python 3.14 classifier. |
| packages/coordinax.curveframes/pyproject.toml | Adds Python 3.14 classifier. |
| packages/coordinax.astro/pyproject.toml | Adds Python 3.14 classifier. |
| packages/coordinax.api/pyproject.toml | Adds Python 3.14 classifier; updates plum-dispatch requirement with Python 3.14 conditional. |
| .github/workflows/ci.yml | Expands CI to test Python 3.14; pins alls-green action by commit SHA. |
d4111a7 to
7aa191e
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #515 +/- ##
==========================================
+ Coverage 92.31% 92.33% +0.02%
==========================================
Files 218 218
Lines 6633 6638 +5
==========================================
+ Hits 6123 6129 +6
+ Misses 510 509 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
| dependencies = [ | ||
| "astropy>=7.0.0", | ||
| "beartype>=0.21.0", | ||
| "coordinax.api", | ||
| "dataclassish>=0.8.0", | ||
| "equinox>=0.13.2", | ||
| "dataclassish>=0.8.0,<0.9.0; python_version == '3.11'", | ||
| "dataclassish>=0.9.0; python_version >= '3.12'", | ||
| "equinox>=0.13.8", | ||
| "jax>=0.6.0", | ||
| "jaxlib>=0.6.0", | ||
| "jaxtyping>=0.3.3", | ||
| "optional-dependencies>=0.3.2", | ||
| "plum>=2.8.0", | ||
| "quax>=0.2.1", | ||
| "quax-blocks>=0.4.0", | ||
| "quaxed>=0.10.4", | ||
| "plum>=2.8.0; python_version < '3.14'", | ||
| "plum>=2.9.0; python_version >= '3.14'", | ||
| "quax>=0.3.2", | ||
| "quax-blocks>=0.4.1", |
| def __call__(self, tau: Any, x: Any = None, /, **kw: Any) -> Any: | ||
| """Apply the operator to the arguments. | ||
|
|
||
| This method calls `coordinax.api.frames.act` to apply the operator. | ||
| If `x` is not provided, `tau` will be passed as `None` to the `act` |
| # CORRECT | ||
| @plum.dispatch | ||
| def process(obj: cxc.AbstractChart, /): # type: ignore[type-arg] | ||
| def process(obj: cxc.AbstractChart, /): |
|
Owee, I'm MrMeeseeks, Look at me. There seem to be a conflict, please backport manually. Here are approximate instructions:
And apply the correct labels and milestones. Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon! Remember to remove the If these instructions are inaccurate, feel free to suggest an improvement. |
No description provided.