bids-specification icon indicating copy to clipboard operation
bids-specification copied to clipboard

[FIX] Data formats clarification

Open nbeliy opened this issue 1 year ago • 6 comments

  • closes #1713

Changes:

In common principles/image format, removed reference to Nifti, and removed Nifti specific content. Rephrased the rest.

In MRI, added Data format section with Nifti requirements.

To do:

  • [x] To add your name, please edit our Contributors wiki and add your name with the type of contribution.

nbeliy avatar Mar 05 '24 13:03 nbeliy

@bids-maintenance

Added explicit mentioning of:

  • incompatibility of ANALYZE 7.5 files with BIDS
  • Concatenation of multi-volume acquisitions (e.g. for 4D fmri)

I'm not sure that this is actual requirements from BIDS, can you confirm or dis-confirm?

nbeliy avatar Mar 05 '24 13:03 nbeliy

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.93%. Comparing base (05f64ed) to head (7f15ae8). Report is 70 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1720   +/-   ##
=======================================
  Coverage   87.93%   87.93%           
=======================================
  Files          16       16           
  Lines        1351     1351           
=======================================
  Hits         1188     1188           
  Misses        163      163           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 05 '24 13:03 codecov[bot]

@Remi-Gau , did you got news from maintainers meeting? Should I go with PET format too?

nbeliy avatar Mar 08 '24 14:03 nbeliy

Added section for PET. Essentially a copy-paste of MRI, with a minimum-recommended version of dcm2niix, and additional sub-sections to better fit into structure of page.

I'll put specific remarks into a review of code.

nbeliy avatar Mar 11 '24 16:03 nbeliy

Hi @Remi-Gau , just to know is there any developments on this merge request?

nbeliy avatar May 14 '24 08:05 nbeliy

Hi @Remi-Gau, just wanted to tell that I will be away until 29 of July.

nbeliy avatar Jul 04 '24 14:07 nbeliy