Skip to content

Improve warning message for short-lived monitored processes#286

Merged
graeme-a-stewart merged 2 commits intoHSF:mainfrom
Siddhant2306:warning-message-fix
Feb 2, 2026
Merged

Improve warning message for short-lived monitored processes#286
graeme-a-stewart merged 2 commits intoHSF:mainfrom
Siddhant2306:warning-message-fix

Conversation

@Siddhant2306
Copy link
Contributor

Summary

This PR updates the warning message shown when a monitored process finishes before the sampling interval elapses.
The previous warning message was slightly unclear for short-lived processes.
This change improves clarity and better guides users toward adjusting the --interval value when needed.

Changes

  • Updated the warning string emitted when the wallclock time is less than the sampling interval

Testing

  • Built the project from a clean build directory
  • Verified the updated warning appears correctly at runtime

Notes

This change is limited to messaging only and does not affect runtime behavior or logic.

siddhant and others added 2 commits January 23, 2026 17:49
Retain specific warning about average statistics.
@graeme-a-stewart
Copy link
Member

Thank you for your suggestion @Siddhant2306. I did think it was still important to include the specific warning about average statistics. I did like adding the suggestion to use the --interval option.

@graeme-a-stewart graeme-a-stewart self-requested a review February 2, 2026 09:40
Copy link
Member

@graeme-a-stewart graeme-a-stewart left a comment

Choose a reason for hiding this comment

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

Approved with the return of the average stats warning line.

@graeme-a-stewart graeme-a-stewart merged commit 4a49553 into HSF:main Feb 2, 2026
8 checks passed
@graeme-a-stewart
Copy link
Member

@all-contributors please add @Siddhant2306 for Doc

@allcontributors
Copy link
Contributor

@graeme-a-stewart

I've put up a pull request to add @Siddhant2306! 🎉

@Siddhant2306
Copy link
Contributor Author

Thank you for your suggestion @Siddhant2306. I did think it was still important to include the specific warning about average statistics. I did like adding the suggestion to use the --interval option.

Thank you for the kind words! I’m glad the change aligned with what you had in mind.
I’m excited to contribute further and would love to stay involved with the project going forward.

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.

2 participants