Skip to content

Add Continuous Receiver Gathers template and grid override - #851

Draft
BrianMichell wants to merge 5 commits into
TGSAI:mainfrom
BrianMichell:add_crg_template
Draft

Add Continuous Receiver Gathers template and grid override#851
BrianMichell wants to merge 5 commits into
TGSAI:mainfrom
BrianMichell:add_crg_template

Conversation

@BrianMichell

@BrianMichellBrianMichell commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Proposes implementation for CRG template and custom override to support epoch-based sensor chunking

EDIT: This PR has been reset to a draft so we can implement better defined templates and re-evaluate the requirements for multi-receiver gathers.

@BrianMichellBrianMichell added bug Something isn't working enhancement New feature or request labels Jul 29, 2026
@codecov

codecovBot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.84615% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.34%. Comparing base (c3ba558) to head (c772040).
⚠️ Report is 250 commits behind head on main.

Files with missing linesPatch %Lines
src/mdio/ingestion/segy/index_strategies.py94.91%1 Missing and 2 partials ⚠️
tests/integration/conftest.py96.59%2 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #851 +/- ##
==========================================
+ Coverage 85.30% 93.34% +8.03% 
==========================================
Files 46 148 +102 Lines 2219 8785 +6566 Branches 306 490 +184 ==========================================
+ Hits 1893 8200 +6307 - Misses 281 508 +227 - Partials 45 77 +32 

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

@andgineer

Copy link
Copy Markdown

@BrianMichell
This branch is cut from main before #852, so anything pinning refs/pull/851/head resolves zarr unbounded, picks up 3.3.0, and dies at import:

pydantic.errors.PydanticSchemaGenerationError: Unable to generate pydantic-core schema
for <class 'zarr.codecs.blosc.BloscCname'>

Could you merge main into the branch? It needs the rebase before it merges anyway..

@BrianMichellBrianMichell removed the bug Something isn't working label Aug 3, 2026
@BrianMichellBrianMichell changed the title Add Continuous Receiver Gathers template and grid override + Re-pin fsspec for regression bugAdd Continuous Receiver Gathers template and grid overrideAug 3, 2026
@BrianMichell
BrianMichell marked this pull request as draft August 5, 2026 15:19
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@BrianMichell@andgineer