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

Add TopoFit #51

Open
hvgazula opened this issue Jul 17, 2023 · 3 comments
Open

Add TopoFit #51

hvgazula opened this issue Jul 17, 2023 · 3 comments

Comments

@hvgazula
Copy link
Contributor

hvgazula commented Jul 17, 2023

The saved model and predict.py scripts can be found at https://github.com/neuroneural/topofit_fork.

@hvgazula hvgazula mentioned this issue Jul 17, 2023
17 tasks
@hvgazula
Copy link
Contributor Author

@satra Topofit uses freesurfer for preprocessing the inputs before inference. Should we include freesurfer as a requirement to enable preprocessing or should we warn the user that the data has to be preprocessed first outside the zoo?

@satra
Copy link
Contributor

satra commented Oct 30, 2023

you can make preprocess an option. presumably you can use the minimal freesurfer for this instead of all of freesurfer. see neurodocker for the min recon-all version. however this can be one of those MVP, MVP+1 things. MVP - don't include, file an issue, include in MVP + 1.

@hvgazula
Copy link
Contributor Author

Makes sense. Thank you.

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