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

Clearing loaded DFK on cleaning the DFK in dfk.cleanup() #3263

Closed
Harichandra-Prasath opened this issue Mar 19, 2024 · 1 comment
Closed

Comments

@Harichandra-Prasath
Copy link
Contributor

Harichandra-Prasath commented Mar 19, 2024

Is your feature request related to a problem? Please describe.
Currently after a DFK is cleaned up by dfk.cleanup() , it cleans the DFK but not clearing the reference of the DFK. So, we have to call parsl.clear() to remove the loaded dfk to load another new config. And if we try to load a new without parsl.clear(), it raises a error indicating there is already a loaded config.

Try

parsl.load()
parsl.dfk().cleanup()
parsl.load()

Describe the solution you'd like
it would be better if dfk.cleanup() can clear the reference as well, as after dfk.cleanup() we cant load new config as well as pushing new tasks.

@benclifford
Copy link
Collaborator

PR #3444 fixes this - thanks for your discussion with me on this @Harichandra-Prasath which helped us understand how this behaviour should change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants