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

Possibly incorrect RepetitionTimeExcitation in multiple sequences #457

Closed
dagra opened this issue Dec 3, 2020 · 2 comments
Closed

Possibly incorrect RepetitionTimeExcitation in multiple sequences #457

dagra opened this issue Dec 3, 2020 · 2 comments

Comments

@dagra
Copy link

dagra commented Dec 3, 2020

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?

@neurolabusc
Copy link
Collaborator

@dagra can you check the latest solution from the development branch. This should resolve your issue.

@dagra
Copy link
Author

dagra commented Dec 4, 2020

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants