-
Notifications
You must be signed in to change notification settings - Fork 34
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
pty_uril
error on Summit
#725
Comments
It is actually not an error, but a warning, which warns about one of the SAGA adaptors (here the SRM adaptor) not being loaded. We just dump the stacktrace from the loading attempt to record info about why it is not loaded. The engine handles failing adaptors gracefully. In this specific case, we did not really use SRM in any of our projects, so it does not cause any follow-up errors. That much for context - any proposal on how to handle this, or if it needs handling? |
Is there a reason because we try to load it if we are not using it? For reporting, it is well identified as a WARNING so that works for me. |
No particular reason: SAGA simply tries to load all adaptors it knows about. At the moment those adaptors are statically defined in the adaptor registry, and we can comment out unsupported plugins. So we can easily disable the SRM adaptor whose state is dubious at best. We do not have the ability to control plugin loading on API level (and probably should not have that). FWIW, RU has a plugin manager which loads plugins by looking at installed (not registered) plugins, and is more flexible - but I never got around switching to it in SAGA. RCT.v2 should use it in several places though (SAGA adaptors, RP schedulers and launch methods, etc). |
Thanks. OK, let's disable SRM then (I doubt anyone will ever use it again anyway) and add a feature 'request' for RCT2, maybe to our design specification document? |
Yes, makes sense - will do! |
Will be resolved by cleaning the config system. |
Stack:
Code:
https://github.com/radical-experiments/summit_jsrun_prte/blob/master/baseline.py
Trace:
The text was updated successfully, but these errors were encountered: