Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename connectomics and deprecate old #820

Merged
merged 9 commits into from
Dec 4, 2023

Conversation

EmmaRenauld
Copy link
Contributor

scil_print_connectivity_filenames --> scil_connectivity_print_filenames
scil_compare_connectivity --> scil_connectivity_compare_populations
scil_compute_connectivity --> scil_connectivity_compute_matrices
scil_reorder_connectivity --> scil_connectivity_reorder_rois
scil_evaluate_connectivity_graph_measures --> scil_connectivity_graph_measures
scil_evaluate_connectivity_pairwise_agreement_measures --> scil_connectivity_pairwise_agreement
scil_normalize_connectivity --> scil_connectivity_normalize
scil_connectivity_math (already ok)
scil_filter_connectivity --> scil_connectivity_filter


NOT CHANGED:

scil_decompose_connectivity ---> It is NOT a connectivity script. It splits a tractogram into bundles (+ preprocessing and cleaning). There is possibly a LOT of overlap with stuff like scil_score_tractogram. To be cleaned.
@frheault We need to manage that some day if you have time to meet with me.

scil_save_connections_from_hdf5 --> The follow-up script.

@pep8speaks
Copy link

pep8speaks commented Nov 30, 2023

Hello @EmmaRenauld, Thank you for updating !

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-12-04 16:10:10 UTC

@arnaudbore
Copy link
Contributor

Build passed ! Good Job 🍻 !

1 similar comment
@arnaudbore
Copy link
Contributor

Build passed ! Good Job 🍻 !

@arnaudbore
Copy link
Contributor

Build passed ! Good Job 🍻 !

1 similar comment
@arnaudbore
Copy link
Contributor

Build passed ! Good Job 🍻 !

@arnaudbore arnaudbore merged commit 09c5817 into scilus:master Dec 4, 2023
@EmmaRenauld EmmaRenauld deleted the rename_connectomics branch December 4, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants