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

Refactor for object oriented models #5

Closed
wants to merge 41 commits into from

Conversation

gkaguirre
Copy link

Massive changes under the hood. Supports multiple models, with each model implemented as a package of object oriented code. The primary solver is now based on fmincon.

External forward model; HRF time-to-peak model
All bounds, seeds, and model computations should be defined in the external model functions.

Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Silence the per-voxel fitting report
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Also fixed a bug in the stimulus representation that was present in the pRF model as well.

Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
Signed-off-by: gkaguirre <[email protected]>
@gkaguirre gkaguirre closed this Oct 22, 2019
@gkaguirre
Copy link
Author

Sorry to keep doing this. I'm going to see about breaking the "fork" link with your repo, given that I'm taking your code in a direction from which it will not be suitable to merge back to your original project.

@gkaguirre gkaguirre deleted the hrfDev branch October 22, 2019 16:59
@kendrickkay
Copy link
Member

kendrickkay commented Oct 27, 2019 via email

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