openmrslab / suspect

70%

LAST BUILD ON BRANCH master

branch: master

CHANGE BRANCH

x

• master
• amares
• mrs_modifications
• v0.2
• docs_numpy_napoleon
• version_numbering
• bug-fix-#17
• DICOM#20
• ENH#21-phase
• denoise_methods_#23
• ENH-#29-BRUKER
• common_phase_adjust
• #33_spectralregistration
• #35_adjust_frequency
• #39_denoising
• mrsobjects_inheritance
• #44_spatialimage
• 45_twix_transform
• spectral-registration-updates
• channel-combination-up
• v0.3
• 52_anonymize_twix
• 56_slice_spectrum
• 50_twix_voxel_size
• 38_twix_te
• 59_tarquin_read_plots
• 63_array_transforms
• 62_voxel_mask
• 66_mask_imagebase
• 67_nifti_io
• 70_volume_directions
• 72_combine_channels_bug
• 74_positive_image_vectors
• 76_autophasing
• 78_sift_match_dtype
• v0.3.2
• v0.3.3
• 80_image_resample
• v0.3.4
• 82_specreg_improvements
• v0.3.5
• 90_image_resample
• 88_siemens_dicom
• 89_changelog
• 0.3.6
• 98_2d_mask
• 100_philipsdt
• 102_twix_anonym
• 87_fitmrsdata
• 85_channelwhitening
• 109_tarquin_wref
• v0.3.7
• 111_tarquinissues
• 107_lcmodeltransform
• v0.3.8
• 115_philips_encoding
• v0.3.9
• 124_ge_pfiles
• 125_multi_vendor_dicom
• 128_channel_combination_axis
• v0.4.0
• 133_rats
• 135_tr
• 137_quant
• v0.4.1
• 139_voxel_axis_bug
• 11_anon_vd
• 143_ima_load_bug
• v0.4.2
• 145_transform_issue
• 146_newfitting
• v0.4.3
• v0.4.4

Build # 11512850045

Build Type

push

github

Committed by web-flow

Commit Message

More general way to loop over dicom group (#175)

Rather than setting a starting point and incrementing by 1 to find CSA header and CSA data, user `dataset.group_dataset(group_number).iterall()`. 

The old method would fail if DICOM anonymisation stripped off the first expected element number, e.g. tag (0029,0010). 

Also fixed CI: Used `python_version` instead of `version`.

Run Details

6 of 6 new or added lines in 1 file covered. (100.0%)

1293 of 1835 relevant lines covered (70.46%)

8.45 hits per line