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

IMP: Enable Zoom In/Out and Dynamic Resizing of scatterplot #34

Open
ChristosMatzoros opened this issue Sep 13, 2024 · 1 comment · May be fixed by #35
Open

IMP: Enable Zoom In/Out and Dynamic Resizing of scatterplot #34

ChristosMatzoros opened this issue Sep 13, 2024 · 1 comment · May be fixed by #35
Assignees

Comments

@ChristosMatzoros
Copy link
Contributor

Current Behavior

Currently, the scatter plot is static and lacks the ability to zoom in on specific areas. This creates a challenge when multiple data points cluster together, especially when they overlap or are displayed in different colors, making it hard to distinguish and accurately identify their locations. Additionally, the plot does not adjust its size according to the window dimensions, leading to usability issues, as users may need to scroll to view the entire plot.

Proposed Behavior

I propose implementing zoom functionality, allowing users to zoom in and out based on the cursor's position. Additionally, the plot should automatically resize based on the window size to improve usability and avoid unnecessary scrolling.

I’m unsure if this feature aligns with the current roadmap or overlaps with any ongoing issues/PRs. I have already implemented a preliminary version of this functionality, but I would appreciate feedback on whether this enhancement is needed and if there is interest in me proceeding with a pull request.

To view the visualization created with this enhancement you can download here. You can open it using qiime tools view viz.qzv to see the proposed behavior. For a better test of this enhancement, please use the values of sample_id for colorBy, and virus_score for both xField and yField.

Thank you

@lizgehret
Copy link
Member

Hey @ChristosMatzoros,

These changes seem reasonable, thanks for submitting this issue! It seems like you've already implemented these changes locally, so feel free to open a PR for me to review if you'd like this to be included in the 2024.10 release - otherwise I most likely won't get to this until after the release.

@lizgehret lizgehret moved this from Needs Triage to Awaiting Info in QIIME 2 - Triage 🚑 Sep 16, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 2024.10 🎃 Sep 17, 2024
@lizgehret lizgehret moved this from Backlog to Needs Review in 2024.10 🎃 Sep 17, 2024
@lizgehret lizgehret moved this from Needs Review to In Development in 2024.10 🎃 Sep 18, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 2025.4 🌻 Oct 3, 2024
@hagenjp hagenjp assigned lizgehret and unassigned ChristosMatzoros Oct 3, 2024
@hagenjp hagenjp removed this from 2024.10 🎃 Oct 3, 2024
@lizgehret lizgehret removed this from 2025.4 🌻 Nov 18, 2024
@lizgehret lizgehret changed the title ENH: scatterplot-2d: Enable Zoom In/Out and Dynamic Resizing of Plot IMP: scatterplot-2d: Enable Zoom In/Out and Dynamic Resizing of Plot Dec 2, 2024
@lizgehret lizgehret changed the title IMP: scatterplot-2d: Enable Zoom In/Out and Dynamic Resizing of Plot IMP: Enable Zoom In/Out and Dynamic Resizing of scatterplot Dec 2, 2024
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.

2 participants