-
Notifications
You must be signed in to change notification settings - Fork 104
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
Move get_machine_name to public API #7965
Conversation
As this function is used outside of ert, it should not be hidden below ensemble_evalutor.config. port_handler.py serves as a placeholder for the code, but the function but exposed and should be used as ert.shared.get_machine.name()
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7965 +/- ##
==========================================
- Coverage 85.81% 85.80% -0.02%
==========================================
Files 378 378
Lines 23073 23079 +6
Branches 629 618 -11
==========================================
+ Hits 19801 19803 +2
+ Misses 3199 3195 -4
- Partials 73 81 +8 ☔ View full report in Codecov by Sentry. |
|
||
from ert.shared import get_machine_name as ert_shared_get_machine_name |
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.
Can this be only from ert.shared import get_machine_name as _get_machine_name
?
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.
Hmm, I think i prefer the explicitness of the long name. Nonetheless, we will change this again in short time when we remove the deprecated (private) API.
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.
Looks good! Just had a small comment about the name
As this function is used outside of ert, it should not be hidden below ensemble_evalutor.config.
port_handler.py serves as a placeholder for the code, but the function but exposed and should be used as ert.shared.get_machine.name()
Issue
Resolves #7888
Approach
🚚
When applicable