Report the number of generated pinch-out connections. #767
Conversation
|
jenkins build this please |
| Opm::OpmLog::info(fmt::format("{} pinch-out connections generated", | ||
| nnc_cells[PinchNNC].size())); |
There was a problem hiding this comment.
So this will print
0 pinch-out connections generated
if there are no additional connections. That's okay, if a little redundant. Worse, however, is that it will print
1 pinch-out connectionS generated
i.e., a plural, for the case of a single pinch-out connection. That's not ideal and there's no reason to do that here because we can easily query the number of connections emit a message in concord in this case.
blattms
force-pushed
the
feature/report-num-pinch-out
branch
from
632a2bd to
ae61c62
Compare
|
Fixed.
Thanks, that was intentional. Somehow I had the wrong URL in my second remote. 🙈 |
|
jenkins build this please |
| @@ -312,6 +314,10 @@ namespace cpgrid | |||
| } | |||
| } | |||
|
|
|||
| Opm::OpmLog::info(fmt::format("{} pinch-out connection{} generated", | |||
| nnc_cells[PinchNNC].size(), | |||
| nnc_cells[PinchNNC].size()>1?"s":"")); | |||
There was a problem hiding this comment.
Tiny nit: The condition needs to be size() != 1 here.
|
Thanks a lot for catching that. Always amazing what I mange to do wrong in the tiniest PRs ever 😅 |
|
jenkins build this please |
No description provided.