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

Fix broken component link across plots #13

Merged
merged 1 commit into from
Oct 1, 2021
Merged

Fix broken component link across plots #13

merged 1 commit into from
Oct 1, 2021

Conversation

eurunuela
Copy link
Collaborator

Closes #12 .

The links were broken because of the ranked sorting of the arrays.

Changes proposed in this pull request:

  • Separate the functions that deal with onClick() events into a scatter-specific and a pie-specific function.
  • Use the index of the selected component on its own plot to find the corresponding label (e.g. ICA_00). Then, find the index corresponding to the label in the other plot(s).

@eurunuela eurunuela merged commit f206777 into ME-ICA:master Oct 1, 2021
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.

Scatter and Pie plots are not always linked correctly
1 participant