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

ivadomed configs for cascaded pipeline #20

Closed
wants to merge 1 commit into from

Conversation

uzaymacar
Copy link
Contributor

@uzaymacar uzaymacar commented Dec 17, 2021

This PR adds the ivadomed config file cascaded.json for running inference on a cascaded pipeline as mentioned in #9.

To Reproduce

0. Make sure you have the following dependencies:

  • sct=5.4.0
  • ivadomed=(git-master-1d81b32c61891e4b5b4f2ffbbb7923e367111dc9*)

1. Clone the repository:

git clone https://github.com/ivadomed/model_seg_ms_mp2rage.git
cd model_seg_ms_mp2rage

2. Clone the latest version of the data:

git clone [email protected]:datasets/basel-mp2rage
cd basel-mp2rage
git annex get .
cd ..

3. Preprocess the data:

sct_run_batch -script preprocessing/preprocess_data.sh -path-data basel-mp2rage -path-output basel-mp2rage-preprocessed-lesionseg -script-args "svm lesionseg" -jobs <JOBS>

4. Run inference wit the cascaded pipeline:

git checkout um/ivadomed_cascaded_pipeline
ivadomed --test -c config/cascaded.json

@uzaymacar
Copy link
Contributor Author

As discussed in the previous meeting, the users of these models want to segment the spinal cord precisely. Therefore, we don't need a cascaded pipeline and I'm closing this PR.

@uzaymacar uzaymacar closed this Jan 16, 2022
@uzaymacar uzaymacar deleted the um/ivadomed_cascaded_pipeline branch January 16, 2022 12:42
@uzaymacar
Copy link
Contributor Author

Returning back to this closed PR, the recent #45 lays out a good motivation for investing in a cascaded model. I will create a new branch and open a new PR to address this shortly.

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.

1 participant