🐛 Correct Threshold Censor Regressor syntax #115
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes
Fixes #114 by @shnizzedy
Description
Updates the Regressor component to:
Censor
fromthreshold
tothresholds
While testing this change, I noticed that some fields aren't included in the generated YAML if they aren't changed from the default. That's not typically an issue since C-PAC will use values from the default pipeline if the values aren't populated, but, for example, the default pipeline doesn't have any values for a custom regressor, so we should
Tests
Try to make a regressor with censoring at https://fcp-indi.github.io/C-PAC_GUI/versions/nightly/browser and in this branch locally (
C-PAC_GUI/README.md
Lines 8 to 10 in 0c942fb
Here are the relevant sections of the YAML files generated during the screen captures below:
before
after
Screencaps
before.mp4
after.mp4
Checklist
Update index.md
).develop
branch of the repository.Developer Certificate of Origin
Developer Certificate of Origin