-
Notifications
You must be signed in to change notification settings - Fork 141
FastMRI dataset onboarding script and detailed examples #444
Conversation
create compressed and uncompressed datasets docu
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.
Great new capabilities added 🚀 And really liked the documentation!
Left some minor comments, feel free to ignore if they don't make sense.
Co-authored-by: melanibe <[email protected]>
At the moment, the
|
correction on my comment above, appears there was an exit(1) in there for debugging purposes that caused the scrip to exit prematurely. Continuing testing with that fixed. |
progress_bar_refresh_rate = container.pl_progress_bar_refresh_rate | ||
if progress_bar_refresh_rate is None and is_azureml_run: | ||
# When running in AzureML, the default progress bar clutters the output files with thousands of lines. | ||
progress_bar_refresh_rate = 50 |
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.
Does this need to be set here? I imagine most containers would override this if necessary (the FastMri container does), and this introduces a small hidden difference between an offline and AzureML run.
This PR adds necessary tooling and examples for running fastMRI reconstruction models.
VarNet
model from the fastMRI github repo