Skip to content

Update Catmull-Rom shader#2786

Merged
johnnovak merged 1 commit intomainfrom
gm/catmull-rom-update-1
Aug 25, 2023
Merged

Update Catmull-Rom shader#2786
johnnovak merged 1 commit intomainfrom
gm/catmull-rom-update-1

Conversation

@GranMinigun
Copy link
Copy Markdown
Contributor

Now it samples in the correct colour space. Also added it to the list of well-behaving shaders, so no more black halo on the edges of the output. Comparison, before and after:

Screenshot_20230825_114850
Screenshot_20230825_115128

Add to the list of known good shaders to get rid of the black halo
on the edges. Enable sampling in correct color space.
@GranMinigun GranMinigun added the shaders Issues related to shaders label Aug 25, 2023
@GranMinigun GranMinigun requested review from johnnovak and kcgen August 25, 2023 07:23
@GranMinigun GranMinigun self-assigned this Aug 25, 2023
Copy link
Copy Markdown
Member

@johnnovak johnnovak left a comment

Choose a reason for hiding this comment

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

Great enhancement @GranMinigun, now it's gamma-correct! 😎

I will experiment with this to use it as the fallback shader for the adaptive CRT shaders instead of sharp. Should only be picked up when we have less than 3 pixels per emulated scanline, so it will give us just the right moderate amount of blur, in theory.

@johnnovak johnnovak merged commit 8270aad into main Aug 25, 2023
@GranMinigun GranMinigun deleted the gm/catmull-rom-update-1 branch August 25, 2023 07:51
@johnnovak johnnovak added the enhancement New feature or enhancement of existing features label Nov 13, 2023
@johnnovak johnnovak added the video Graphics and video related issues label Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement of existing features shaders Issues related to shaders video Graphics and video related issues

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants