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.

Sign up for GitHub

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

Jump to bottom

New stable release #624

Merged
merged 68 commits into from
Aug 4, 2022
Merged

New stable release #624

merged 68 commits into from
Aug 4, 2022

Conversation

neurolabusc
Copy link
Collaborator

neurolabusc and others added 30 commits October 12, 2021 09:07
@neurolabusc
XA30 behavior (#236)
d32ef79
@neurolabusc
additional ZIP2 detection mechanism (https://github.com/rordenlab/dcm…
ebf7435 
…2niix/issues)
@neurolabusc
Temporal discrepancy warning (#560)
c7ee1e5
@yhuang43 @ningfei
1. Implement a wrapper class to interface with dcm2niix functions.
da316b0 
2. The wrapper class provides interface to convert dicom in mgz orientation.
3. The wrapper class and dcm2niix functions are compiled into libdcm2niixfs.a
   with -DUSING_DCM2NIIXFSWRAPPER -DUSING_MGH_NIFTI_IO.
4. When using libdcm2niixfs.a, instead of outputting .nii, *.bval, *.bvec to disk,
   nifti header, image data, TDICOMdata, & TDTI information are saved in MRIFSSTRUCT struct.
5. If libdcm2niixfs.a is compiled with -DUSING_MGH_NIFTI_IO, the application needs to link with nifti library.
@yhuang43 @ningfei
set target_include_directories, target_link_libraries for dcm2niixfs
7a3c46d 
for -DZLIB_IMPLEMENTATION=Cloudflare -DUSE_JPEGLS=ON -DUSE_OPENJPEG=ON
@yhuang43 @ningfei
1. add user control option BLD_DCM2NIIXFSLIB=ON
93d899c 
2. add install command for libdcm2niixfs.a
3. don't build libdcm2niixfs.a if USE_OPENJPEG OR USE_TURBOJPEG OR USE_JASPER is ON
@ningfei
Refine CMake support for building libdcm2niixfs.
a133247 
Based on PR #559. Also formatting the .cpp/.h files.
@ningfei
Merge pull request #562 from ningfei/development
5664008 
Add option to build libdcm2niixfs.
@jonclayden
Resolving additional printf() calls
08d6e78
@jonclayden
Removing fence for pure comment lines
83608ae
@jonclayden
std::min and std::max don't always do the job because argument types …
49c76eb 
…differ
@neurolabusc
Merge pull request #570 from jonclayden/divest-0.10.2
9897372 
Changes for divest 0.10.2
@neurolabusc
Experimental support for 1-bit DICOM (#572)
53281c4
@neurolabusc
Detect echo number for Siemens XA that omit 0018,0086 (#568)
57a3e9d
@neurolabusc
Treat CPS manufacturer as Siemens (#576)
41416b1
@neurolabusc
Experimental support for Mediso DWI
06a1f97
@neurolabusc
Reduce compiler warnings, TR check now is aware of multiple averages …
2057d89 
…(Mediso)
@neurolabusc
HRRT sequence details (#577)
a0b7510
@neurolabusc
Reorder PT volumes by ascending FrameReferenceTime (#577)
822a4e1
@fangq
Export NeuroJSON JSON/binary-JSON based JNIfTI files (jnii/bnii)
fd10424
@fangq
fix an indentation
3d8eeaf
@fangq
add new units related to JNIfTI support
36ffc8c
@fangq
use -z to decide if jnii and bnii need compression, use zlib as ziptype
ae2a103
@fangq
add _ArrayOrder_ field because nifti is column-major
aa50fbe
@fangq
remove memory leak due to duplicated deflateInit
cc514c7
@neurolabusc @fangq
JNIfTI support modular (cmake -DUSE_JNIFTI=OFF ..)
d8fb0b5
@fangq
update cmake setting to use renamed cJSON.cpp
5d22eb4
@fangq
force base64 encoding when compression is not specified
b3c4bfe
@ningfei
Merge pull request #580 from NeuroJSON/development
5b50245 
Add support to export NeuroJSON (.jnii and .bnii) files.
@NPann
fix new line for warning message
496ccc3
ningfei and others added 28 commits April 12, 2022 21:57
@ningfei
Bump CMake minimum required version.
7d12f98
@mr-jaemin
GE bias value for b-value
61397b5
@neurolabusc
Merge pull request #603 from mr-jaemin/development
4ab28e2 
GE bias value for b-value
@neurolabusc
Ignore Original Attributes SQ (#599)
c6b80d3
@neurolabusc
Ignore ReferencedImageEvidenceSQ
89c2844
@neurolabusc
Remove test
9169c7f
@neurolabusc
XA30 can include CSA header
5d096d1
@neurolabusc
Report DwellTime for Siemens XA (#240)
0af5159
@neurolabusc
Better Siemens XA support (#606)
b29ea93
@neurolabusc
Flipping Y also flips sign of determinant
6e51368
@neurolabusc
Ignore non-spatial physio data (#606)
dea3fb9
@neurolabusc
scan options is long string, fix bvec rejection (#606)
98ccdbe
@neurolabusc
Change scanOptions
f8f421e
@neurolabusc
Verbose scan options (issue 606)
b71678a
@neurolabusc
Philips slice timing notes
c3b4fef
@neurolabusc
GE sequence details (#608)
8788733
@neurolabusc
Store GE ShimSetting as array (#608)
0191d51
@neurolabusc
Add AcquisitionVoxelSize tag for Siemens ASL (#608)
8f66f1e
@neurolabusc
AcquisitionVoxelSize before any interpolation or resampling within re…
f9cc431 
…construction or image processing
@neurolabusc
shims are signed (#608)
a716dc6
@neurolabusc
PostLabelDelay for XA30, FrameDuration is only for 4D datasets (#616)
63c76f2
@neurolabusc
Reset PET values for classic DICOMs (#616)
3583fcc
@neurolabusc
Siemens XA30 ASL parameters and ImageTypeText 0021,1175
4f89073
@neurolabusc
Update notes
26a74a5
@neurolabusc
Issue 618 (#618)
160aaa2
@mr-jaemin
GE Direct field mapping (TE1/TE2) (#617)
625a247
@mr-jaemin
GE Direct field mapping (TE1/TE2) (#617)
cfa4b98
@neurolabusc
Merge pull request #619 from mr-jaemin/development
f2ceab6 
Thank @mr-jaemin  this will close GE Direct field mapping (TE1/TE2) (#617)
@neurolabusc neurolabusc merged commit bb3a6c3 into master Aug 4, 2022
@luissantosHCIT luissantosHCIT mentioned this pull request Apr 10, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@neurolabusc @yhuang43 @ningfei @jonclayden @fangq @NPann @mr-jaemin