Hyperplane update

[frheault]

Quick description

Attempt at having our improved RADTRACT Hyperplane subdivision !

https://www.ncbi.nlm.nih.gov/pmc/articles/PMC10246281/
...

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

[pep8speaks]

Hello @frheault, Thank you for updating !

• In the file scilpy/tractanalysis/bundle_operations.py:

Line 93:80: E501 line too long (80 > 79 characters)

• In the file scilpy/tractanalysis/distance_to_centroid.py:

Line 57:1: W293 blank line contains whitespace
Line 81:80: E501 line too long (81 > 79 characters)
Line 184:80: E501 line too long (80 > 79 characters)
Line 200:80: E501 line too long (80 > 79 characters)
Line 265:35: E203 whitespace before ','
Line 292:1: W293 blank line contains whitespace
Line 300:1: W293 blank line contains whitespace
Line 339:37: E127 continuation line over-indented for visual indent
Line 340:37: E127 continuation line over-indented for visual indent
Line 342:21: E127 continuation line over-indented for visual indent
Line 352:22: E128 continuation line under-indented for visual indent
Line 354:80: E501 line too long (83 > 79 characters)
Line 371:22: E128 continuation line under-indented for visual indent
Line 383:22: E128 continuation line under-indented for visual indent
Line 395:22: E128 continuation line under-indented for visual indent
Line 398:22: E128 continuation line under-indented for visual indent
Line 416:41: E128 continuation line under-indented for visual indent
Line 418:21: E128 continuation line under-indented for visual indent
Line 419:1: W293 blank line contains whitespace
Line 421:5: E303 too many blank lines (2)
Line 428:5: E303 too many blank lines (2)

• In the file scripts/scil_bundle_label_map.py:

Line 205:80: E501 line too long (80 > 79 characters)
Line 246:80: E501 line too long (92 > 79 characters)
Line 266:80: E501 line too long (81 > 79 characters)
Line 300:80: E501 line too long (84 > 79 characters)
Line 314:80: E501 line too long (81 > 79 characters)

Comment last updated at 2024-12-12 18:45:11 UTC

[codecov]

Codecov Report

Attention: Patch coverage is 76.87688% with 77 lines in your changes missing coverage. Please review.

Project coverage is 58.89%. Comparing base (d28df8c) to head (dc5a0cf).
Report is 46 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (d28df8c) and HEAD (dc5a0cf). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (d28df8c)	HEAD (dc5a0cf)
unittests	2	1

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #1034       +/-   ##
===========================================
- Coverage   72.79%   58.89%   -13.91%     
===========================================
  Files         451      651      +200     
  Lines       24647    30705     +6058     
  Branches     3380     3460       +80     
===========================================
+ Hits        17942    18083      +141     
- Misses       5271    11173     +5902     
- Partials     1434     1449       +15     

Components	Coverage Δ
Scripts	57.26% <58.33%> (-18.20%)	⬇️
Library	61.34% <85.77%> (-7.91%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[frheault]

All good (I think), probably ready for merge?

[AntoineTheb]

@frheault there are still a lot of pep8 issues to fix. Also the main can probably be split into multiple functions to make the code much easier to read and review.

[AntoineTheb]

Round of review. Mostly needs more comments to explain what the code is doing and why, and needs more cleanup. Functions need to be split into smaller chunks and pep8 issues need to be fixed.

[CHrlS98]

Thanks for the extensive documentation. Single comment on the references in your main script below.

[AntoineTheb]

LGTM

[AntoineTheb]

@arnaudbore @frheault we should keep this in mind as it will come back to bite us