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

Turn on Work Queue coprocess mode #2779

Merged
merged 18 commits into from
Jul 19, 2023
Merged

Turn on Work Queue coprocess mode #2779

merged 18 commits into from
Jul 19, 2023

Conversation

benclifford
Copy link
Collaborator

This should probably be the preferred way of using the work queue executor, as it can speed things up incredibly on shared filesystem supercomputers, and even on my laptop can reduce WQ pytest run time from around 550s to 350s.

Type of change

  • New feature (non-breaking change that adds functionality)
    ?

This should probably be the preferred way of using the work queue executor,
as it can speed things up incredibly on shared filesystem supercomputers, and
even on my laptop can reduce WQ pytest run time from around 550s to 350s.
@benclifford
Copy link
Collaborator Author

this hung in CI trying to run WQ tests... further investigation needed so converting to draft

@benclifford benclifford marked this pull request as draft June 29, 2023 05:04
@benclifford
Copy link
Collaborator Author

in CI runinfo logs:

003$ cat ./submit_scripts/parsl.WorkQueueExecutor.block-0.1688148747.9381468.sh.err
sh: 1: parsl/executors/workqueue//parsl_coprocess.py: not found

so this looks like a packaging/install issue of some kind

@benclifford
Copy link
Collaborator Author

subsequent fiddling has shown there are some subtleties with the PATH mechanism set up by the Makefile... i'll make a separate PR for this first

@benclifford
Copy link
Collaborator Author

see PR #2804 for build PATH simplifications which should make coprocesses work

@benclifford benclifford marked this pull request as ready for review July 19, 2023 15:39
@benclifford benclifford merged commit 3fc9293 into master Jul 19, 2023
@benclifford benclifford deleted the benc-wq-coprocess-on branch July 19, 2023 19:46
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

Successfully merging this pull request may close these issues.

2 participants