Skip to content

New: "NoSuccess" decorator processor#75

Open
matthias-mayr wants to merge 1 commit intoRobotLabLTH:masterfrom
matthias-mayr:pr_no_success
Open

New: "NoSuccess" decorator processor#75
matthias-mayr wants to merge 1 commit intoRobotLabLTH:masterfrom
matthias-mayr:pr_no_success

Conversation

@matthias-mayr
Copy link
Member

This decorator can be useful as part of a recovery process that should still produce a failure feedback to the parent.

This decorator can be useful as part of a recovery process
that should still produce a failure feedback to the parent.
@matthias-mayr
Copy link
Member Author

If this is merged, it should be mentioned here in the wiki as well.

@frovida
Copy link

frovida commented Jul 6, 2023

Nothing against. I would take the chance to give a more clear naming: FailureIsSuccess, and SuccessIsFailure. Borrowed from pytree https://py-trees.readthedocs.io/en/devel/decorators.html . If you have spare time, you could consider to put in a base class to avoid repeat 70% of the code.

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