Skip to content

Conversation

@EmmaRenauld
Copy link
Contributor

Nicer print info! Fixes issue #1122

nii.gz file:

**************************************
File name:    t1.nii.gz
**************************************
  Dimensions: [137 166 133]
  Voxel size: [1 1 1]
  Datatype: <class 'numpy.uint8'>
  Orientation: ('L', 'P', 'S')
  Afine (vox2rasmm):
[[ -1.    0.    0.  -20.5]
 [  0.   -1.    0.  -20.5]
 [  0.    0.    1.    7.5]
 [  0.    0.    0.    1. ]]

mgz file:

**************************************
File name:    t1.mgz
**************************************
  Dimensions: [137 166 133   1]
  Voxel size: [1. 1. 1.]
  Datatype: <class 'numpy.uint8'>
  Orientation: ('L', 'P', 'S')
  Afine (vox2rasmm):
[[ -1.    0.    0.  -20.5]
 [  0.   -1.    0.  -20.5]
 [  0.    0.    1.    7.5]
 [  0.    0.    0.    1. ]]

.trk file:

**************************************
File name:    tracto.trk
**************************************
  Dimensions: [138 166 134]
  Voxel size: [1. 1. 1.]
  Datatype: float32
  Orientation: ('L', 'P', 'S')
  Afine (vox2rasmm):
[[ -1.    0.    0.  -19.5]
 [  0.   -1.    0.  -21.5]
 [  0.    0.    1.    9.5]
 [  0.    0.    0.    1. ]]

tck file:

**************************************
File name:    tracto.tck
**************************************
  Dimensions: (138, 166, 134)
  Voxel size: (1.0, 1.0, 1.0)
  Datatype: Float32LE
  Orientation: ('R', 'A', 'S')
  Afine (vox2rasmm):
[[1. 0. 0. 0.]
 [0. 1. 0. 0.]
 [0. 0. 1. 0.]
 [0. 0. 0. 1.]]

@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

❌ Patch coverage is 79.31034% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.57%. Comparing base (df80acf) to head (2508d3a).
⚠️ Report is 30 commits behind head on master.

❌ Your patch status has failed because the patch coverage (79.31%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (72.57%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1272      +/-   ##
==========================================
+ Coverage   72.45%   72.57%   +0.11%     
==========================================
  Files         293      294       +1     
  Lines       25203    25328     +125     
  Branches     3530     3546      +16     
==========================================
+ Hits        18262    18382     +120     
  Misses       5450     5450              
- Partials     1491     1496       +5     
Flag Coverage Δ
smoketests 69.76% <79.31%> (+0.13%) ⬆️
unittests 13.28% <0.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Scripts 75.45% <79.31%> (+0.16%) ⬆️
Library 69.14% <ø> (+0.04%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arnaudbore arnaudbore enabled auto-merge October 31, 2025 17:18
@arnaudbore arnaudbore added this pull request to the merge queue Oct 31, 2025
Merged via the queue into scilus:master with commit dd6b5dc Oct 31, 2025
3 checks passed
@EmmaRenauld EmmaRenauld deleted the fix_issue1122 branch October 31, 2025 18:16
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

Successfully merging this pull request may close these issues.

2 participants