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

local distances return faulty output #21

Closed
sina-mansour opened this issue Mar 6, 2019 · 1 comment · Fixed by #38
Closed

local distances return faulty output #21

sina-mansour opened this issue Mar 6, 2019 · 1 comment · Fixed by #38
Assignees
Labels
Milestone

Comments

@sina-mansour
Copy link

Hi,

I'm using the gdist.local_gdist_matrix function to get the all to all local distances. There are two problems however that seem wrong with the output:

  1. The output matrix is not symmetric. However, if a distance from i to j is reported, I don't see why the distance from j to i shouldn't be reported. Am I right to expect a symmetric matrix as an output?

  2. Some of the elements of the output matrix are larger than the max_distance provided as an input. Is there something wrong going on in the code?

Thanks.

ayan-b added a commit to ayan-b/tvb-geodesic that referenced this issue Jun 7, 2020
@liadomide
Copy link
Member

@sina-mansour

  1. From our tests the symmetry is broken only with a small difference (<1e-6) due to precision issues. Have you observed something different ?
  2. Indeed, the max_distance was not considered before. It should be now (after we will merge TVB-2719 Add local_gdist_matrix tests and fix Bug 21 #38 from @ayan-b ). But we still consider acceptable if max_distance will be overcome with a very small Epsilon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants