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

Add API call for pairwise distance computation #52

Closed
ayan-b opened this issue Jul 22, 2020 · 0 comments · Fixed by #54
Closed

Add API call for pairwise distance computation #52

ayan-b opened this issue Jul 22, 2020 · 0 comments · Fixed by #54
Assignees
Milestone

Comments

@ayan-b
Copy link
Member

ayan-b commented Jul 22, 2020

Related to #51.

A new API for this:

Let's say that I have 180 points. On the first iteration I take the first one and compute the distance to the other 179, on the following iteration i take the second one and compute the distance to the next 178, and so on.

In particular, for n points we would consider the 1st one as source and rest of the (n - 1) ones as target and, then 2nd one as source and (n - 2) as target and so on...

@ayan-b ayan-b self-assigned this Jul 24, 2020
@ayan-b ayan-b added this to the GSOC2020 milestone Jul 24, 2020
ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jul 24, 2020
ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jul 25, 2020
ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jul 25, 2020
Check if distances is of type sparse matrix. Also
add API doc in README.rst and add docstring to
distance_matrix_of_selected_points

Related to the-virtual-brain#52
ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jul 25, 2020
Check if distances is of type sparse matrix. Also
add API doc in README.rst and add docstring to
distance_matrix_of_selected_points

Related to the-virtual-brain#52
@liadomide liadomide changed the title Add API for new use case Add API call for pairwise distance computation Jul 25, 2020
ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jul 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant