-
Notifications
You must be signed in to change notification settings - Fork 27
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
Testing/add etp and add configuration #20
Conversation
…f those test results
…arge. Needs investigation
…nto testing/add_ETP_and_add_configuration
…d_ETP_and_add_configuration
Hey! Several questions: What is the reason for now initializing the code for every voxel?: |
Moreover, the Bayesian approach needs an initial guess that is reasonable. This is now taken from the segmented fit. If the segmented fit is broken, it is expected that the Bayesian will not work well. :) |
I think for the Bayesian wrapper you can remove line 77 " fit_results=fit_results+(1,)" |
Unfortunately, I cannot run the tests on segmented fit without the json file (which is now being ignored by .gitignore). |
…b.com/OSIPI/TF2.4_IVIM-MRI_CodeCollection into testing/add_ETP_and_add_configuration
I think I've fixed the errors in segmented and bayesian. Still confused about the initialisation. |
…uration Testing/add etp and add configuration
No description provided.