-
Notifications
You must be signed in to change notification settings - Fork 108
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
Do not use IPC protocol for everest #9621
base: main
Are you sure you want to change the base?
Conversation
CodSpeed Performance ReportMerging #9621 will not alter performanceComparing Summary
|
@@ -306,7 +306,7 @@ def main(): | |||
optimization_callback=partial(_opt_monitor, shared_data=shared_data), | |||
) | |||
if run_model.ert_config.queue_config.queue_system == QueueSystem.LOCAL: | |||
evaluator_server_config = EvaluatorServerConfig() | |||
evaluator_server_config = EvaluatorServerConfig(use_ipc_protocol=False) |
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.
What does this do?
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.
With zmq, you can opt for using IPC (inter process communication) protocol, which is faster than TCP. This of-course is possible only when using LOCAL_DRIVER. On the other hand, I don't think that it solves the issue.
@@ -19,7 +19,7 @@ def main(argv): | |||
arg_parser.add_argument("--fail", type=str) | |||
options, _ = arg_parser.parse_known_args(args=argv) | |||
|
|||
if options.fail in os.getcwd(): | |||
if options.fail is not None and options.fail in os.getcwd(): |
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 am unsure this change is needed.
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.
Right, but otherwise this will fail all the time (ie. each realization), at least that's what I got when testing it
Issue
Resolves #9620
Approach
WIP
(Screenshot of new behavior in GUI if applicable)
git rebase -i main --exec 'pytest tests/ert/unit_tests -n logical -m "not integration_test"'
)When applicable