-
Notifications
You must be signed in to change notification settings - Fork 42
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
Shepp logan phantom #202
Shepp logan phantom #202
Conversation
… with **kwargs, training_root and validation_root not required now for train - some datasets don't require saved data.
…to allow for file filtering, minor fixes
Codecov Report
@@ Coverage Diff @@
## main #202 +/- ##
==========================================
+ Coverage 80.69% 80.79% +0.10%
==========================================
Files 71 72 +1
Lines 5096 5317 +221
==========================================
+ Hits 4112 4296 +184
- Misses 984 1021 +37
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, but please add references where possible, e.g. in simulating a sensitivity map.
direct train
,direct predict
andpython3 projects/predict_val.py
to not necessarily require path to data as some datasets don't require it -build_dataset_from_input
relies on**kwargs
now