[tuflow] Implement H timeseries processor pipeline #39
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…mplement-hprocessor-for-time-series-data
|
@codex review |
|
To use Codex here, create a Codex account and connect to github. |
Chain-Frost
merged commit d5a6bc7
into
work-on-qprocessor-that-could-break-stuff
1 check passed
Chain-Frost
pushed a commit
that referenced
this pull request
Dec 2, 2025
* change json to suit suggested config update by codex * Simplify Q timeseries processing (#21) * Merge remote-tracking branch 'origin/main' into work-on-qprocessor-that-could-break-stuff * Add stub TUFLOW H/V processors (#27) * Refactor POMM abs max derivation (#26) * Document BaseProcessor CSV loaders (#24) * Remove obsolete TUFLOW output file (#25) * Consolidate PO combination logic (#23) * [processors] Refresh CSV ingestion helpers (#28) * Enhance shared TUFLOW CSV readers * Merge branch 'work-on-qprocessor-that-could-break-stuff' into codex/refactor-data-processing-classes-and-methods * [core] Drive POMM processing from config (#29) * Refine POMM config handling * tweaks * Merge branch 'work-on-qprocessor-that-could-break-stuff' into codex/extend-_load_configuration-in-pommprocessor * pandas-stubs * might have broken it more * Refactor timeseries processors for Q and V (#32) not checked * Simplify TUFLOW processing metadata (#33) * Document TUFLOW processor workflow (#34) * Document TUFLOW processor extension workflow (#35) * Document POMM and PO workflows * Merge remote-tracking branch 'origin/main' into work-on-qprocessor-that-could-break-stuff * Merge remote-tracking branch 'origin/main' into work-on-qprocessor-that-could-break-stuff * Merge branch 'main' into work-on-qprocessor-that-could-break-stuff * 2025-11-02 progress * Merge branch 'main' into work-on-qprocessor-that-could-break-stuff * Added configuration guards inside BaseProcessor._load_configuration * Document processor import flow and config hooks (#36) * Document TUFLOW maximum and timeseries processors (#37) * Merge pull request #38 from Chain-Frost:codex/update-documentation-for-tuflow-processors [docs] Expand POMM and PO processor workflow docs * [tuflow] Implement H timeseries processor pipeline (#39) * Add H timeseries processor and coverage * tweaks * Merge branch 'work-on-qprocessor-that-could-break-stuff' into codex/implement-hprocessor-for-time-series-data * Merge branch 'main' into work-on-qprocessor-that-could-break-stuff * extra test data - TUFLOW_Example_Model_Dataset * feat: Add TUFLOW processors, results validation and datatypes configuration, and robustness tests, while updating .gitignore. * add eof processor * Merge branch 'main' into work-on-qprocessor-that-could-break-stuff * first part logging tweaks * logging improvements - remove duplicate module * progress on tests, but there is a lot more work to do * progress on tests, black formatter * more testing progress * basic mcp, doesn't really do anything though * cache dir * test tweaks * add 1D gpkg results and tlf extample datasets * tidy testing * processor file path log text shortened * tidy up tuflow python scripts and some that are ss now * more test coverage expansion * Merge remote-tracking branch 'origin/main' into work-on-qprocessor-that-could-break-stuff * more test stuff * add garbage collector, other tweaks * standardisation of wrappers for tuflow functions * resolve test failures, still need to check with actual RORB data
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Testing
https://chatgpt.com/codex/tasks/task_e_68cb0fe1595c832e9c9223168ac09a92