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 2D/3D view artifacts on view's border when using fractional zoom #8369

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Dec 9, 2024

Related

What

Wasn't able to repro the issue in the first place though - tried fractional zoom and a contrast heavy background to no avail

  • confirm this fixes it

@Wumpf Wumpf added 🪳 bug Something isn't working 🔺 re_renderer affects re_renderer itself 📺 re_viewer affects re_viewer itself include in changelog labels Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
e4d6bf8 https://rerun.io/viewer/pr/8369 +nightly +main

Note: This comment is updated whenever you push a commit.

@Wumpf Wumpf requested a review from emilk December 10, 2024 13:13
Copy link
Member

@abey79 abey79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm deeply unqualified with that gpu minutiea, but since this one has been waiting for quite some time, I'm gonna apply my stamp of approval to give a confident @Wumpf license for merging :)

Code lgtm. Maybe there could be a comment somewhere to explain the phenomenon and its fix?

@Wumpf
Copy link
Member Author

Wumpf commented Dec 16, 2024

@abey79 did you have repro of the issue earlier? I failed to reproing it so I'd like to confirm whether this fixes it

@abey79 abey79 merged commit 02a8757 into main Dec 16, 2024
31 of 32 checks passed
@abey79 abey79 deleted the andreas/fix-border-artifacts-fractional-zoom branch December 16, 2024 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working include in changelog 🔺 re_renderer affects re_renderer itself 📺 re_viewer affects re_viewer itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rendering glitch in 3D view with selection
2 participants