Skip to content

[LGR] Pass in writeInitial an NNCCollection to allow output of the 3 NNC types#6972

Open
aritorto wants to merge 1 commit intoOPM:masterfrom
aritorto:bridgeNNCs
Open

[LGR] Pass in writeInitial an NNCCollection to allow output of the 3 NNC types#6972
aritorto wants to merge 1 commit intoOPM:masterfrom
aritorto:bridgeNNCs

Conversation

@aritorto
Copy link
Copy Markdown
Member

@aritorto aritorto commented Mar 31, 2026

This PR invokes the bridge between simulation NNCs and NNCCollection for output files, implemented in OPM/opm-common#5089.

With this change, all three NNC types get written out.

To test this functionality, run the simulation in serial mode with:

--parsing-strictness=low
--enable-ecl-output=true

Draft mode since it seems that some throw statements might need to be removed in NNCCollection::getGlobalNNC()

@aritorto aritorto marked this pull request as draft March 31, 2026 10:43
@aritorto aritorto added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Mar 31, 2026
@aritorto aritorto marked this pull request as ready for review March 31, 2026 13:16
@aritorto aritorto marked this pull request as draft March 31, 2026 13:17
@aritorto aritorto marked this pull request as ready for review March 31, 2026 13:53
@aritorto
Copy link
Copy Markdown
Member Author

jenkins build this serial please

@aritorto aritorto marked this pull request as draft March 31, 2026 15:08
@aritorto aritorto marked this pull request as ready for review April 9, 2026 08:19
@aritorto
Copy link
Copy Markdown
Member Author

aritorto commented Apr 9, 2026

jenkins build this serial please

@aritorto aritorto force-pushed the bridgeNNCs branch 3 times, most recently from d34218c to 668a31a Compare April 13, 2026 16:01
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.

1 participant