Skip to content

Commit 5ffa272

Browse files
tlambert03claude
andauthored
feat: v2 rewrite, new iterator pattern (take 3) (#251)
* feat: add decoupled v2 multi-axis sequence module This adds the v2 module with self-contained implementations of: - AxisIterable: Generic base for axis iteration - MultiAxisSequence: Generic multi-axis iteration - TimePlan, ZPlan, GridPlan: Axis-specific implementations - MDASequence: V1-compatible wrapper with v2 architecture Key design decisions: - V2 plan classes are independent (no inheritance from v1) - Shared types (MDAEvent, Channel, Axis) imported from v1 - Clean separation enables independent evolution Test status: - All v1 tests pass (171) - Core v2 tests pass (129) - Position sub-sequence tests need additional work 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fix: add filterwarnings for v2 position sequence deprecation The test was failing because pytest is configured with filterwarnings=["error"], which converts the DeprecationWarning from Position.__new__ (when handling legacy Position.sequence attribute) into an error. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * migration guide * move enums * reuse test cases * moving some v1 stuff * fix: skip tests on Windows due to tcl issues * bump min pydantic * bump again * skip windows plot * undo v1 folder * remove __eq__ * remove doc * refactor: update MDAEvent contribution structure to include is_relative flag * fix pydantic issue * fix docs * fix tcl * ignore * ignore again * fix duration * remove dupe * add grid from poly * switch runner * add classmethods --------- Co-authored-by: Claude <noreply@anthropic.com>
1 parent 7724185 commit 5ffa272

42 files changed

Lines changed: 6729 additions & 1333 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ jobs:
7070

7171
test-dependents:
7272
name: ${{ matrix.repo }}
73-
runs-on: macos-13
73+
runs-on: macos-latest
7474
env:
7575
UV_MANAGED_PYTHON: "1"
7676
UV_NO_SYNC: "1"

.pre-commit-config.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ repos:
2727
- types-PyYAML
2828
- pydantic >=2
2929
- numpy >=2
30+
- shapely
3031

3132
- repo: local
3233
hooks:

0 commit comments

Comments
 (0)