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

Remove duplicate MPI launcher abstractions #3532

Open
benclifford opened this issue Jul 18, 2024 · 0 comments
Open

Remove duplicate MPI launcher abstractions #3532

benclifford opened this issue Jul 18, 2024 · 0 comments

Comments

@benclifford
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

PR #3016 introduced a second MPI launcher abstraction inside the High Throughput Executor, distinct from the long-existing parsl.launchers abstraction.

Mostly this was driven by the new usages of launchers in #3016, although extremely similar to earlier usage, being different enough that the existing launcher API could not support this new use.

The launcher abstraction should be fixed to support all these use cases.

Describe the solution you'd like

one launcher abstraction

Describe alternatives you've considered

status quo: forcing all future launcher implementers, maintainers and users to deal with two MPI launcher abstractions

Additional context

In #3016 (review) I merged #3016 with severe reservations including the subject of this current issue. This issue is to turn that in-passing comment into a numbered issue.

@benclifford benclifford changed the title Remove duplicate mi launcher abstractions Remove duplicate MPI launcher abstractions Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant