facebookresearch / fastMRI

A large-scale dataset of both raw MRI measurements and clinical MRI images.
https://fastmri.org
MIT License
1.28k stars 371 forks source link

Issue with argparse in PyTorch Lightning v2 #322

Open stperrakis opened 9 months ago

stperrakis commented 9 months ago

Hello,

I've been updating the code in line with the changes introduced in PyTorch Lightning v2's documentation. I'm working to ensure that all modifications align correctly and the code functions as intended.

Once I've finalized the changes, would you be open to a pull request?

Thank you.

mmuckley commented 9 months ago

Hello @stperrakis

Oh yes, definitely open to a pull request! Primary thing is that it needs to conform to the formatting standards and behaviors of the existing repository so that users familiar with the old code need to change as little as possible (or ideally, don't need to change anything).