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

[ENH] Add soft distance for each filter #719

Merged
merged 4 commits into from
May 23, 2023

Conversation

arnaudbore
Copy link
Contributor

No description provided.

@arnaudbore arnaudbore requested a review from frheault May 15, 2023 18:39
@arnaudbore
Copy link
Contributor Author

Build passed ! Good Job 🍻 !

@frheault
Copy link
Member

I would prefer if this was a parameter in filter_grid_roi() (or same for bdo) and keeping the parameter --soft_distance, this way it would work for drawn roi, slices, and atlas the same way.

It would be possible to use --soft_distance_roi and --soft_distance_bdo, since bdo is in mm (I think)

@arnaudbore
Copy link
Contributor Author

I would like to be able to add distance specifically to each filter we have and not something overall specific to a type.
We have a couple of examples where you only want to have this distance applied to the a specific ROI and not all of them.
I will make it easier to read but do you think we should also have --soft_distance_roi and --soft_distance_bdo to be able to apply an overall distance ?

@frheault
Copy link
Member

frheault commented May 16, 2023

Ok what about:
We keep --soft_distance_roi and --soft_distance_bdo for --drawn_roi and --bdo
And we add a dilation last for --atlas_roi, if it finishes with include or exclude, it means no dilation and if it finishes with a number that number is the number of passes.

the nargs could be '+' with a check on the number accepted
--atlas_roi ROI_NAME ID MODE CRITERIA (DIST)
4 or 5 (and if 5, ends with integer)

OR:

We remove --soft_distance entirely and we do the optional logic describe above for EVERYTHING
like: --z_plane PLANE MODE CRITERIA (DIST)
And we count the option, 3 or 4 (and if 4, must be integer)

@arnaudbore
Copy link
Contributor Author

@arnaudbore
Copy link
Contributor Author

@arnaudbore
Copy link
Contributor Author

Build passed ! Good Job 🍻 !

@arnaudbore arnaudbore merged commit 06a2715 into scilus:master May 23, 2023
@arnaudbore arnaudbore deleted the enh_scil_filtering branch January 30, 2024 21:58
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