Skip to content

Commit

Permalink
apply_olo not used anywhere. Deleting
Browse files Browse the repository at this point in the history
  • Loading branch information
EmmaRenauld committed Feb 3, 2025
1 parent 9d050c9 commit 0dd6725
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 29 deletions.
28 changes: 3 additions & 25 deletions scilpy/connectivity/matrix_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,27 +43,6 @@ def compute_olo(array):
return perm


def apply_olo(array, perm):
"""
Apply the permutation from compute_RCM.
Parameters
----------
array: ndarray (NxN)
Sparse connectivity matrix.
perm: ndarray (N,)
Permutations for rows and columns to be applied.
Returns
-------
ndarray (N,N)
Reordered array.
"""
if array.ndim != 2:
raise ValueError('RCM can only be applied to 2D array.')
return array[perm].T[perm]


def apply_reordering(array, ordering):
"""
Apply a non-symmetric array ordering that support non-square output.
Expand Down Expand Up @@ -101,13 +80,12 @@ def apply_reordering(array, ordering):
def evaluate_graph_measures(conn_matrix, len_matrix, avg_node_wise,
small_world):
"""
toDo Finish docstring
Parameters
----------
conn_matrix: np.ndarray
2D matrix of connectivity weights.
len_matrix: np.ndarray of shape ??
2D matrix of connectivity weights. Typicaly a streamline count matrix.
len_matrix: np.ndarray
2D matrix of bundle lengths.
avg_node_wise: bool
If true, return a single value for node-wise measures.
small_world: bool
Expand Down
1 change: 1 addition & 0 deletions scilpy/connectivity/tests/test_connectivity.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ def test_compute_triu_connectivity_from_labels():


def test_compute_connectivity_matrices_from_hdf5():
# ToDo. We will have to create a test hdf5.
pass


5 changes: 1 addition & 4 deletions scilpy/connectivity/tests/test_matrix_tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,7 @@


def test_compute_olo():
pass


def test_apply_olo():
# Simple and basically only using hierarchy's function. Not testing.
pass


Expand Down

0 comments on commit 0dd6725

Please sign in to comment.