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.

Sign up for GitHub

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

Jump to bottom

BaseRestartWorkChain: add enable/disable handlers input port #3773

Closed
sphuber opened this issue Feb 20, 2020 · 0 comments · Fixed by #3786
Closed

BaseRestartWorkChain: add enable/disable handlers input port #3773

sphuber opened this issue Feb 20, 2020 · 0 comments · Fixed by #3786
Assignees
@sphuber
Labels
priority/important topic/workflows type/accepted feature approved feature request
Milestone
v1.1.1

Comments

@sphuber
Copy link
Contributor

sphuber commented Feb 20, 2020 •
edited
Loading

The process_handler should only be applicable to class methods of BaseRestartWorkChain sub classes, which means they will also be active by default. We should add an enabled keyword to the process_handler decorator to change this default behavior and also add an input that allows to either enable or disable specific handlers per instance.
@sphuber sphuber added this to the v1.1.1 milestone Feb 20, 2020
@sphuber sphuber self-assigned this Feb 20, 2020
@sphuber sphuber changed the title BaseRestartWorkChain: make register_process_handler class method and add handler exclude/include input BaseRestartWorkChain: add enable/disable handlers input port Feb 21, 2020
This was referenced Feb 21, 2020
Merged
Merged
@sphuber sphuber modified the milestones: v1.1.1, v1.1.2 Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sphuber sphuber

Labels
priority/important topic/workflows type/accepted feature approved feature request
Projects
None yet
Milestone
v1.1.1
Development

Successfully merging a pull request may close this issue.

BaseRestartWorkChain: add method to enable/disable process handlers sphuber/aiida-core
1 participant
@sphuber