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

Package and document better #82

Open
jcohenadad opened this issue Apr 12, 2024 · 1 comment
Open

Package and document better #82

jcohenadad opened this issue Apr 12, 2024 · 1 comment
Assignees

Comments

@jcohenadad
Copy link
Member

jcohenadad commented Apr 12, 2024

Based on our meeting with a neuroradiologist today, as well as some confusion around the usage of this fonction (see eg: sct-pipeline/spine-park#17 (comment)), we need to improve the documentation of this function (eg: 10min video to-the-point).

Also, I suggest to make it a pip-installable package (if that's possible without too much trouble).

@valosekj
Copy link
Member

Based on our meeting with a neuroradiologist today, as well as some confusion around the usage of this fonction (see eg: sct-pipeline/spine-park#17 (comment)), we need to improve the documentation of this function (eg: 10min video to-the-point).

Agree! I will focus on this after ISMRM.

Also, I suggest to make it a pip-installable package (if that's possible without too much trouble).

Yes, this is definitely an option. Or we can reconsider again the integration of the 'manual-correction' module directly into SCT (as discussed in the past in #3353). By doing this, users would have to download this repo as an external dependency. Moreover, all scripts related to "manual correction workflow" such as sct_run_batch, sct_qc and potentially manual_correction would be integrated directly into SCT.

@valosekj valosekj self-assigned this Apr 12, 2024
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

No branches or pull requests

2 participants