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

Auto3DSeg AutoRunner Fire CLI Chaining Fails #7167

Closed
drbeh opened this issue Oct 27, 2023 · 0 comments · Fixed by #7168
Closed

Auto3DSeg AutoRunner Fire CLI Chaining Fails #7167

drbeh opened this issue Oct 27, 2023 · 0 comments · Fixed by #7168
Assignees
@drbeh

Comments

@drbeh
Copy link
Member

drbeh commented Oct 27, 2023

Describe the bug
Auto3DSeg AutoRunner uses Fire to generate CLI. Fire supports chaining of multiple methods in a single command. However, it fails for AutoRunner and they cannot chained. I realized that the AutoRunner methods does not return self, which is a requirements for Fire CLI chaining to work.
@drbeh drbeh self-assigned this Oct 27, 2023
@drbeh drbeh mentioned this issue Oct 27, 2023
Merged
1 task
drbeh added a commit that referenced this issue Oct 30, 2023
@drbeh
Enable Chaining in Auto3DSeg CLI (#7168)
1c17f0e 
Fixes #7167

### Description
Make all the setting methods in `AutoRunner` to return `self` to enable
chaining in cli.

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).

---------

Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@drbeh drbeh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enable Chaining in Auto3DSeg CLI drbeh/MONAI
1 participant
@drbeh