Open VincentBeaud opened 5 months ago
Hello @VincentBeaud, Thank you for updating !
scripts/scil_fibertube_tracking.py
:Line 183:80: E501 line too long (85 > 79 characters)
scripts/scil_tractogram_filter_collisions.py
:Line 44:80: E501 line too long (81 > 79 characters)
scripts/tests/test_fibertube_tracking.py
:Line 80:80: E501 line too long (84 > 79 characters)
@mdesco et @CHrlS98 Voici la grosse PR! Il me reste un peu de job à faire alors elle est en draft. Aussi, je viens de faire plusieurs petits changements et je n'ai pas retesté chaque script. Je vais donc le faire maintenant en créant un fichier de test pour chaque.
Je vais aussi rajouter une description détaillée dès que j'aurai terminé ça.
Attention: Patch coverage is 59.80498%
with 371 lines
in your changes missing coverage. Please review.
Project coverage is 68.51%. Comparing base (
1d6f848
) to head (4adc5eb
).
@mdesco
- Should we rename all files that contain ft or fibertube. If not, how do we make sure our users know what ft means?
Yes I think we should replace ft
by fibertube
. We will want our example in the documentation to clearly explain what fibertubes are too.
New script names: scil_tractogram_filter_collisions.py -> unchanged scil_ft_visualize_collisions.py -> scil_viz_tractogram_collisions.py scil_ft_fibers_metrics.py -> deleted. Code is in "scil_tractogram_filter_collisions.py" scil_ft_tracking.py -> scil_fibertube_tracking.py scil_ft_reconstruction_metrics.py -> scil_fibertube_score_tractogram.py
Quick description
Please include a summary of the changes and the related issue(s) or improvement(s). Please also include relevant motivation and context. List any dependencies that are required for this change if needed.
...
Type of change
Check the relevant options.
Provide data, screenshots, command line to test (if relevant)
...
Checklist