Skip to content

Add optional canonical dataset CRS string - #878

Merged
BrianMichell merged 3 commits into
TGSAI:mainfrom
BrianMichell:feat/dataset-crs
Sep 17, 2026
Merged

BrianMichell merged 3 commits into
TGSAI:mainfrom
BrianMichell:feat/dataset-crs

Conversation

@BrianMichell

@BrianMichell BrianMichell commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Implements a canonical CRS string that lives in the dataset metadata level. Also implements a way to safely update this information for existing MDIO files.

Closes #794

Survey CRS lives on the template, not SEG-Y ingest.
In-place updates share a Zarr attr helper so v2
consolidated metadata cannot erase sibling keys.

Co-authored-by: Cursor <cursoragent@cursor.com>
@BrianMichell BrianMichell added the enhancement New feature or request label Sep 17, 2026
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.28%. Comparing base (c3ba558) to head (31cc364).
⚠️ Report is 257 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #878      +/-   ##
==========================================
+ Coverage   85.30%   93.28%   +7.97%     
==========================================
  Files          46      152     +106     
  Lines        2219     8870    +6651     
  Branches      306      501     +195     
==========================================
+ Hits         1893     8274    +6381     
- Misses        281      517     +236     
- Partials       45       79      +34     

☔ 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.

@BrianMichell
BrianMichell merged commit 8b75e9c into TGSAI:main Sep 17, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant