Skip to content

Fix bluriness when there is multiple different outputs scaling values#46

Merged
any1 merged 2 commits intoany1:masterfrom
stacyharper:wbarraco/push-vmprxsuskvpy
Feb 12, 2025
Merged

Fix bluriness when there is multiple different outputs scaling values#46
any1 merged 2 commits intoany1:masterfrom
stacyharper:wbarraco/push-vmprxsuskvpy

Conversation

@stacyharper
Copy link
Contributor

@stacyharper stacyharper force-pushed the wbarraco/push-vmprxsuskvpy branch from ef15eab to 500a648 Compare February 10, 2025 08:21
@stacyharper
Copy link
Contributor Author

I'm a bit annoyed by this link2, but is there a better way?

Copy link
Owner

@any1 any1 left a comment

Choose a reason for hiding this comment

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

I believe "Fix blurriness when there are multiple output scaling values" is the more conventional way of saying this?

@stacyharper stacyharper force-pushed the wbarraco/push-vmprxsuskvpy branch 2 times, most recently from 37c5c4c to 5ee08a6 Compare February 10, 2025 10:44
@stacyharper
Copy link
Contributor Author

Maybe it's better to just mark the outputs instead using a variable named something like has_surface.

I reworked those patch this way, and applied your recommandations.

To do so, we filter outputs that the surface has not entered when
computing the max scale on the list.
This is available on version 6 or later. So we keep our own code to be
retro-compatible.
@stacyharper stacyharper force-pushed the wbarraco/push-vmprxsuskvpy branch from 5ee08a6 to 61b3086 Compare February 10, 2025 10:49
@any1 any1 merged commit e2f6ab7 into any1:master Feb 12, 2025
@any1
Copy link
Owner

any1 commented Feb 12, 2025

Thanks!

@stacyharper stacyharper deleted the wbarraco/push-vmprxsuskvpy branch February 26, 2025 19:31
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.

2 participants