-
Notifications
You must be signed in to change notification settings - Fork 58
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
T2* mapping model #445
base: usrjson
Are you sure you want to change the base?
T2* mapping model #445
Conversation
Codecov Report
@@ Coverage Diff @@
## usrjson #445 +/- ##
===========================================
- Coverage 47.40% 46.30% -1.10%
===========================================
Files 260 282 +22
Lines 8647 10370 +1723
===========================================
+ Hits 4099 4802 +703
- Misses 4548 5568 +1020
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@jvelazquez-reyes for tests we may consider a really small dataset for this model. |
Is there any available on OSF? |
Not yet, but if you can crop the images to a smaller extent, we can add it. |
Great! Let's make sure that the test actually runs for this model. Then we'll merge usejson first to master. |
@jvelazquez-reyes see this error. The test is failing because the filenames you provide for the example dataset do not match those in Current file names: Required file names: That's how genBatch infers which file to load to which field. Can you also update the |
I see, thanks! What names would you prefer to keep? |
It is not really up to my preference :) Please see the wiki docs: https://github.com/qMRLab/qMRLab/wiki/Guideline:-Uploading-sample-data |
Cool, I like |
Purpose
T2* mapping model working with default parameters
Open Questions and Pre-Merge TODOs
Use github checklists. When solved, check the box and explain the answer.
Review that changed source files/lines are related to the pull request/issue
If any files/commits were accidentally included, cherry-pick them into another branch.
Review that changed source files/lines were not accidentally deleted
Fix appropriately if so.
Test new features or bug fix
If not implemented/resolved adequately, solve it or inform the developer by requesting changes in your review.
Preferably, set breakpoints in the locations that the code was changed and follow allong line by line to see if the code behaves as intended.
Manual GUI tests (general)