Skip to content

WIP: refactor of vecs, operators#473

Merged
nstarman merged 8 commits into
GalacticDynamics:mainfrom
nstarman:vecs/refactor
Apr 16, 2026
Merged

WIP: refactor of vecs, operators#473
nstarman merged 8 commits into
GalacticDynamics:mainfrom
nstarman:vecs/refactor

Conversation

@nstarman
Copy link
Copy Markdown
Contributor

@nstarman nstarman commented Nov 17, 2025

Not sure if this is a good idea. It may be much better for jax.
The representation-type information can be static and now there's only one Vector type (plus the KinematicSpace collection object).

@nstarman nstarman added this to the v1.0.0 milestone Nov 17, 2025
@nstarman nstarman force-pushed the vecs/refactor branch 2 times, most recently from 21afbec to d9a1185 Compare November 23, 2025 05:39
@nstarman nstarman force-pushed the vecs/refactor branch 2 times, most recently from 4737af8 to c51ef7d Compare December 8, 2025 01:31
@github-actions github-actions Bot added 📝 Add / update documentation Add or update documentation. ✅ Add / update / pass tests Add, update, or pass tests. ✨ Introduce new features Introduce new features. labels Dec 24, 2025
@github-actions github-actions Bot added the 🔧 Add / update configuration Add or update configuration files. label Jan 6, 2026
@github-actions github-actions Bot added the 🙈 Add / update .gitignore Add or update a .gitignore file. label Jan 23, 2026
@nstarman nstarman changed the title WIP: refactor of vecs WIP: refactor of vecs, operators Jan 25, 2026
@nstarman nstarman force-pushed the vecs/refactor branch 2 times, most recently from e9b59aa to 4d36085 Compare February 27, 2026 20:33
@github-actions github-actions Bot added the 👷 Add / update CI build system Add or update CI build system. label Feb 27, 2026
@nstarman nstarman force-pushed the vecs/refactor branch 5 times, most recently from cf5c965 to 3f4ca77 Compare March 7, 2026 21:56
@github-actions github-actions Bot added the 💥 Introduce breaking changes Introduce breaking changes. label Mar 7, 2026
@nstarman nstarman force-pushed the vecs/refactor branch 5 times, most recently from e4cc24c to 0e6ff2a Compare March 8, 2026 06:27
@nstarman nstarman force-pushed the vecs/refactor branch 7 times, most recently from 5f377a0 to f6d8573 Compare March 23, 2026 13:30
@nstarman nstarman force-pushed the vecs/refactor branch 10 times, most recently from e9515b7 to af17f05 Compare March 27, 2026 23:51
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 28, 2026

Codecov Report

❌ Patch coverage is 97.33333% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.07%. Comparing base (ea44648) to head (69629e7).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
packages/coordinax.api/src/coordinax/api/charts.py 81.81% 4 Missing ⚠️
...rdinax.astro/src/coordinax/astro/_setup_package.py 88.88% 3 Missing ⚠️
...frames/src/coordinax/curveframes/_setup_package.py 87.50% 2 Missing ⚠️
...kages/coordinax.api/src/coordinax/api/manifolds.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #473      +/-   ##
==========================================
- Coverage   98.44%   92.07%   -6.38%     
==========================================
  Files         139      215      +76     
  Lines        4129     6322    +2193     
==========================================
+ Hits         4065     5821    +1756     
- Misses         64      501     +437     

☔ View full report in Codecov by Sentry.
📢 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.

Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
@nstarman
Copy link
Copy Markdown
Contributor Author

Part 1 of a few PRs to get this complete.

@adrn
Copy link
Copy Markdown
Contributor

adrn commented Apr 16, 2026

Woo!!!

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

Labels

👷 Add / update CI build system Add or update CI build system. 🔧 Add / update configuration Add or update configuration files. 📝 Add / update documentation Add or update documentation. 🙈 Add / update .gitignore Add or update a .gitignore file. ✅ Add / update / pass tests Add, update, or pass tests. 💥 Introduce breaking changes Introduce breaking changes. ✨ Introduce new features Introduce new features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants