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

BaseRestartWorkChain let handler_overrides also override priority #5545

Closed
sphuber opened this issue May 29, 2022 · 0 comments · Fixed by #5546
Closed

BaseRestartWorkChain let handler_overrides also override priority #5545

sphuber opened this issue May 29, 2022 · 0 comments · Fixed by #5546
Assignees
Milestone

Comments

@sphuber
Copy link
Contributor

sphuber commented May 29, 2022

Currently the handler_overrides input allows to enable or disable certain error handlers that are defined on a work chain. There might be use cases where users may want to change the priority of handlers. Currently the form of the input is as follows

handler_overrides = {
    'handler_one': False,
    'handler_two': True,
}

We could add support for more keywords, but would have to go through a deprecation pathway:

handler_overrides = {
    'handler_one': {'enable': False, 'priority': 100},
    'handler_two': {'enable': True, 'priority': 200},
}
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.

1 participant