Skip to content

Improve reservoir coupling log readability#6781

Open
hakonhagland wants to merge 1 commit intoOPM:masterfrom
hakonhagland:fix_rc_log_msgs
Open

Improve reservoir coupling log readability#6781
hakonhagland wants to merge 1 commit intoOPM:masterfrom
hakonhagland:fix_rc_log_msgs

Conversation

@hakonhagland
Copy link
Contributor

Builds on #6775, which should be merged first.

Cleaned up reservoir coupling logging to improve readability.

  • Added Logger::debug() method for debug-level logging
  • Added formatDays() helper for human-readable time display
  • Change verbose MPI communication messages from info to debug level so they only appear in .DBG file, not .PRT file or stdout
  • Add info-level message showing sync step coordination: "Choosing next sync time between master and N active slave process(es): X days"

@hakonhagland hakonhagland added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Jan 29, 2026
@hakonhagland
Copy link
Contributor Author

jenkins build this please

@hakonhagland hakonhagland marked this pull request as draft January 29, 2026 11:51
@hakonhagland
Copy link
Contributor Author

Putting this in draft mode until #6775 has been merged

@hakonhagland
Copy link
Contributor Author

jenkins build this please

- Add Logger::debug() method for debug-level logging
- Add formatDays() helper using unit::day for human-readable time display
- Change verbose MPI communication messages from info to debug level
  so they only appear in .DBG file, not .PRT file or stdout
- Add info-level message showing sync step coordination:
    "Choosing next sync time between master and N active slave process(es): X days"
@hakonhagland hakonhagland marked this pull request as ready for review February 5, 2026 15:46
@hakonhagland
Copy link
Contributor Author

jenkins build this please

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