Skip to content

Conversation

@aritorto
Copy link
Member

@aritorto aritorto commented Dec 17, 2025

Bug found while running case with CARFIN block taking the entire grid (global refinement) enabling ecl output file.

This PR updates EclipseGrid construction to use the level-0 grid’s logicalCartesianSize() , which is necessary for correct EGRID file generation for CpGrid level grids with LGRs. This method is called here.

@aritorto aritorto added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Dec 17, 2025
@aritorto aritorto changed the title Use level zero logicalCartesianSize [bugfix] Use level zero logicalCartesianSize Dec 17, 2025
@aritorto
Copy link
Member Author

jenkins build this serial please

@blattms blattms added bug and removed bug labels Dec 19, 2025
@aritorto aritorto marked this pull request as draft December 19, 2025 16:54
@aritorto aritorto changed the title [bugfix] Use level zero logicalCartesianSize [bugfix] CpGrid::logicalCartesianSize() returns level zero grid logCartSize Dec 22, 2025
@aritorto aritorto marked this pull request as ready for review December 22, 2025 10:44
@aritorto
Copy link
Member Author

jenkins build this serial please

@aritorto
Copy link
Member Author

jenkins build this opm-tests=1343 opm-simulators=6304 please

1 similar comment
@aritorto
Copy link
Member Author

jenkins build this opm-tests=1343 opm-simulators=6304 please

@blattms
Copy link
Member

blattms commented Dec 30, 2025

jenkins build this please

@blattms blattms merged commit 4bdb5ff into OPM:master Dec 30, 2025
2 checks passed
@aritorto aritorto deleted the useL0LogCartSize branch December 30, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants