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 dendrogram view #127

Merged
merged 4 commits into from
Dec 5, 2024
Merged

Fix dendrogram view #127

merged 4 commits into from
Dec 5, 2024

Conversation

stefanhahmann
Copy link
Collaborator

This PR fixes bugs that could occur, if multiple DendrogramViews have been opened by the user.

If they were closed, they were not actually garbage collected and listeners were still active leading to bugs in some situations.

… instance

* Remove the listeners from the components on close()
* Reasoning: before this commit, if the user had closed the JFrame, the instance of it was still kept, since the DendrogramView object was never cleaned up. Thus, also the listeners were still unnecessarily active, which lead to bugs in certain situations
* Without this, the clustering may lead to wrong results, since it operates on a branch graph that potentially is outdated.
Copy link

sonarqubecloud bot commented Dec 5, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
75.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@stefanhahmann stefanhahmann merged commit c809e53 into master Dec 5, 2024
5 of 6 checks passed
@stefanhahmann stefanhahmann deleted the fix-dendrogram-view branch December 5, 2024 13:27
@stefanhahmann stefanhahmann self-assigned this Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant