-
Notifications
You must be signed in to change notification settings - Fork 29
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
Continue working on moving Challenge data preparation utilities to SIRF. #1266
Conversation
I won't have time to review this today, but I think you've addressed my main concerns, so go ahead when you're ready. Given the tortuous history here, I suggest a squash-merge, but please edit the commit message such that it not just a list of all the attempts, but an overall conclusion. (Note that I see you're adding modality, which is great, so add that to the CHANGES.md). thanks! |
looks like you still need to remove examples/Python/PET/test_data_preparation.py |
Changes in this pull request
Testing performed
Related issues
PR #1241
Checklist before requesting a review
Contribution Notes
Please read and adhere to the contribution guidelines.
Please tick the following: