-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add debug info to clipboard capability #8639
Conversation
3adcee8
to
d749a37
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8639 +/- ##
==========================================
- Coverage 90.88% 90.87% -0.02%
==========================================
Files 339 339
Lines 20933 21020 +87
==========================================
+ Hits 19026 19101 +75
- Misses 1907 1919 +12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
4459b5b
to
567816a
Compare
9e4acd3
to
f1a878a
Compare
273b2b0
to
cdbff4f
Compare
kv["Config file"] = self._config_file | ||
kv["Ensemble path"] = self.config.ens_path | ||
kv["Ensemble size"] = str(self._model.ensemble_size) | ||
kv["Max running"] = str(self.config.queue_config.queue_options.max_running) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could possibly list all the values in the queue_options class, or only non-defaulted (or if the latter is tricky, all the non-None values).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opted for the non-None variant. Looking at LSF-options there were mostly those variants. Other systems like Torque or Slurm had some extras, but we can rather expand on that if need be.
e311bb1
to
017c669
Compare
017c669
to
33229bf
Compare
Good to go, just some rebasing (and Everest is maybe fixed by now?) |
33229bf
to
ad3374b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go!
Issue
Resolves #8582
Pressing
F1
or using the button in run_dialog will yield this markdown in your clipboard(screenshot and printout are from different runs)
When applicable