Skip to content

Conversation

@frheault
Copy link
Member

@frheault frheault commented Aug 28, 2025

Quick description

Working prototype test with stride [-1,2,3,4]
To be continued:

  • scil_aodf_metrics.py: Computes metrics from asymmetric ODFs.
  • scil_bingham_metrics.py: Computes metrics from Bingham distribution fits.
  • scil_btensor_metrics.py: Computes metrics using the DIVIDE method with b-tensor
    encodings.
  • scil_dki_metrics.py: Computes Diffusion Kurtosis Imaging (DKI) metrics.
  • scil_dti_metrics.py: Computes Diffusion Tensor Imaging (DTI) metrics.
  • scil_dwi_to_sh.py: Computes SH coefficients from raw DWI signal.
  • scil_fodf_to_bingham.py: Fits a Bingham distribution to fODF lobes.
  • scil_gradients_apply_transform.py: Transforms b-vectors using an affine/rigid
    transformation.
  • scil_gradients_modify_axes.py: Flips or swaps axes of the gradient sampling matrix.
  • scil_mrds_metrics.py: Computes metrics for each Multi-Resolution Discrete-Search
    (MRDS) solution.
  • scil_qball_metrics.py: Computes Constant Solid Angle (CSA) or Analytical Q-ball model
    metrics.
  • scil_sh_convert.py: Converts SH files between different bases.
  • scil_sh_fusion.py: Merges a list of Spherical Harmonics files.
  • scil_sh_to_aodf.py: Estimates asymmetric ODFs (aODFs) from a spherical harmonics
    image.
  • scil_sh_to_rish.py: Computes Rotationally Invariant Spherical Harmonics (RISH)
    features.
  • scil_sh_to_sf.py: Samples SF values from a Spherical Harmonics signal.
  • scil_tracking_local_dev.py: Local streamline HARDI tractography.
  • scil_viz_bingham_fit.py: Visualizes a 2D slice of a Bingham volume.
  • scil_viz_dti_screenshot.py: Registers DWI to a template for screenshots.
  • scil_viz_gradients_screenshot.py: Visualizes directions on a sphere.

...

Type of change

Check the relevant options.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Provide data, screenshots, command line to test (if relevant)

...

Checklist

  • My code follows the style guidelines of this project (run autopep8)
  • I added relevant citations to scripts, modules and functions docstrings and descriptions
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I moved all functions from the script file (except the argparser and main) to scilpy modules
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@codecov
Copy link

codecov bot commented Aug 28, 2025

Codecov Report

❌ Patch coverage is 85.24590% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.56%. Comparing base (8de071b) to head (44d602c).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1220      +/-   ##
==========================================
+ Coverage   72.54%   72.56%   +0.02%     
==========================================
  Files         291      291              
  Lines       25072    25105      +33     
  Branches     3505     3507       +2     
==========================================
+ Hits        18188    18217      +29     
- Misses       5394     5396       +2     
- Partials     1490     1492       +2     
Components Coverage Δ
Scripts 75.31% <78.37%> (-0.01%) ⬇️
Library ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AntoineTheb
Copy link
Contributor

AntoineTheb commented Dec 2, 2025

Wait so ... how were non-RAS strides handled by scilpy before ?

edit: answered offline

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