Skip to content

Add RiaSummaryAddressCollectionTools utility module#13628

Merged
magnesj merged 1 commit intodevfrom
#13513-make-drag-drop-more-robust
Feb 18, 2026
Merged

Add RiaSummaryAddressCollectionTools utility module#13628
magnesj merged 1 commit intodevfrom
#13513-make-drag-drop-more-robust

Conversation

@magnesj
Copy link
Member

@magnesj magnesj commented Feb 17, 2026

Refactor summary address collection logic into a new utility module, RiaSummaryAddressCollectionTools, with functions for object identification, curve definition generation, duplicate filtering, and address-to-case/ensemble mapping. Update RimSummaryPlot to use these utilities, improving modularity and maintainability. Add comprehensive unit tests for the new module.

Closes #13513

@magnesj magnesj requested review from jonjenssen and kriben February 17, 2026 08:35
Copy link
Collaborator

@kriben kriben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Refactor summary address collection logic into a new utility module, RiaSummaryAddressCollectionTools, with functions for object identification, curve definition generation, duplicate filtering, and address-to-case/ensemble mapping. Update RimSummaryPlot to use these utilities, improving modularity and maintainability. Add comprehensive
unit tests for the new module.

Adds test for mock summary case
@magnesj magnesj force-pushed the #13513-make-drag-drop-more-robust branch from 1c8e3aa to 20beaa2 Compare February 18, 2026 09:06
@magnesj magnesj merged commit 8e40005 into dev Feb 18, 2026
19 checks passed
@magnesj magnesj deleted the #13513-make-drag-drop-more-robust branch February 18, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Summary Plot: "Append Curves For Well" not working for another Summary Case

2 participants

Comments