Skip to content
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

309 bug matlab ci is broken #311

Open
wants to merge 23 commits into
base: last-running-matlab
Choose a base branch
from

Conversation

bendhouseart
Copy link
Contributor

@bendhouseart bendhouseart commented Jun 7, 2024


📚 Documentation preview 📚: https://pet2bids--311.org.readthedocs.build/en/311/

bendhouseart and others added 23 commits May 10, 2024 15:56
…d ecat2nii.m

* update python to go along with matlab changes in 9beee53
* moved int testing to this repo
* added function for first_middle_last_frames_to_text for arrays since they're different than cell arrays.
* fixed min/max, need to check difference between outputs before final
* Update ecat2nii.py (#308)
* Use more idiomatic way of constructing an affine
* Remove unnecessary header zeroing, use helper methods where appropriate
* add ad-hoc final test script
* run black, add conditional for testing
* Apply suggestions from code review
   fixed % and added rms
   Update ecat_testing/read_matlab_nii.py
   Co-authored-by: Chris Markiewicz <[email protected]>
* bump version before merge to main

---------

Co-authored-by: Chris Markiewicz <[email protected]>
* update python conversion script for newer phantoms
* Add 3DRP recon; fix phantom conversions


---------

Co-authored-by: Anthony Galassi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants