Skip to content
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

reconsider complex env-var dict mechanism for associating autopara info with specific function calls #318

Open
bernstei opened this issue Jun 14, 2024 · 0 comments

Comments

@bernstei
Copy link
Contributor

The mechanism that's currently documented for passing remoteinfo to autopara ops in docs/source/overview.queued.md is quite complex (in its most general form). Even though I came up with it, I've mostly stopped using it.

Instead, I've switched to explicitly passing RemoteInfo objects (or dicts) in the actual function call, but obtaining those by importing from another file, to separate the task and the details for parallelizing on any given machine. E.g. I have a python file called remote_details.py, which contains

MD_operation = {"one_hpc_machine": { dict with remoteinfo stuff for that machine },
                "other_hpc_machine": {dict with remoteinfo stuff fot this other machine}
               }

which I then import, and when I call the autopara MD function I pass remoteinfo = remote_details.MD_operation["one_hpc_machine"].

I think this is versatile enough, and simpler from the view of the wfl implementation. We should consider removing the env-var based code, and revising the docs as needed.

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

No branches or pull requests

1 participant