-
Notifications
You must be signed in to change notification settings - Fork 229
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
Possibly incorrect RepetitionTimeExcitation in multiple sequences #457
Comments
neurolabusc
added a commit
that referenced
this issue
Dec 4, 2020
@dagra can you check the latest solution from the development branch. This should resolve your issue. |
I checked the latest solution and the problem seems resolved. Great, thanks! |
yarikoptic
added a commit
to neurodebian/dcm2niix
that referenced
this issue
Apr 6, 2021
* commit 'v1.0.20200427-97-g0587941': (22 commits) Overflow for Siemens data [missing protocol name] (rordenlab#466) MacOS notarization, minor tweaks Increase details for series stacking, enhance merge override mode (rordenlab#463) Bump version date Retain Philips Scaling Values for images where scaling does not vary but volumes must be separated (rordenlab#461) Support huge PAR/REC files (rordenlab#460) Prevent dti4D overflow Fix PAR/REC ADC detection (rordenlab#462) DICOM field map calibrated in Hz given name _fieldmaphz (rordenlab#455) PAR/REC field map calibrated in Hz given name _fieldmaphz (rordenlab#455) Ignore LocationsInAcquisition (0020,1002) if number of slices is not evenly divisible with this value (rordenlab#459) Convert DICOM series where bit allocated (0028,0100) varies across slices (rordenlab#458) Clear RepetitionTimeExcitation (rordenlab#457) leak (rordenlab#454) Single file mode memory allocation (rordenlab#454) When -n is specified, only save BIDS for requested series (rordenlab#453) Use 1st study time if multiple times provided. Apple. M1. use C++ not. C Fix CMake for Apple Silicon (note similar change required for CloudFlare zlib) Only report b-value for GE data with 0043,1039 if EPI (0018,9018) (rordenlab#449) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
As I was checking the field
RepetitionTimeExcitation
in the latest release, I noticed that if I convert a lot of sequences at once and for one of them the field is filled, then for some of the remaining series the field is also filled with the same value (which seems incorrect), but if I convert each sequence separately, then the field is filled only for the one that seems correct and it's missing from the others. I also checked the latest development branch and the output is the same. So, is it possible that an initialization is missing somewhere or I get something wrong?The text was updated successfully, but these errors were encountered: