libretro / slang-shaders

223 stars 93 forks source link

CRT Guest Advanced NTSC shader settings are displaying incorrectly #508

Open zltbugs opened 10 months ago

zltbugs commented 10 months ago

When attempting to change the connection type within the CRT Guest Advanced NTSC shader settings, the options displayed are less than what is showing in the help.

Within the ntsc-pass1.slang file, the line that lists the Values, Svideo, Composite, RF does not have those options listed in the shader parameters within Retroarch.

Also there are long dashes that are also listed, while other sections are listed as [SECTION NAME].

/shaders/shaders_slang/crt/shaders/guest/advanced/ntsc/ntsc-pass1.slang

#pragma parameter ntsc-row1 "------------------------------------------------" 0.0 0.0 1.0 1.0
pragma parameter quality "Values: Svideo = 0.0 | Composite = 1.0 | RF = 2.0" 0.0 0.0 1.0 1.0
pragma parameter cust_fringing "NTSC Fringing Value" 1.0 0.0 5.0 0.1
pragma parameter cust_artifacting "NTSC Artifacting Value" 1.0 0.0 5.0 0.1
pragma parameter ntsc-row2 "------------------------------------------------" 0.0 0.0 1.0 1.0

Screenshot from 2023-11-08 16-24-00

hizzlekizzle commented 10 months ago

less than what is showing in the help

Which 'help' are you referring to here?

zltbugs commented 10 months ago

Which 'help' are you referring to here?

The scrolling text that displays the options for each setting. Will post a screen capture when I have an opportunity to get back on that machine.

zltbugs commented 10 months ago

Added a screen shot of the issue.

hizzlekizzle commented 10 months ago

I think that was just an oversight on guest.r's part. That is, I suspect he change the values without changing the help text during one of the updates.

nfp0 commented 10 months ago

That looks like a mistake indeed. Unless he removed one of the modes, which I doubt.

nfp0 commented 8 months ago

I've noticed the pragma parameter quality "Values: Svideo = 0.0 | Composite = 1.0 | RF = 2.0" 0.0 0.0 1.0 1.0 setting has no effect at all. It worked fine before.

hizzlekizzle commented 8 months ago

Correct. It seems guest.r completely removed the quality presets. Dunno why.

guestrr commented 7 months ago

"Presets" are still intact, but now like both Artifacting and Fringing are to be set at the same time to 0.0 for Svideo, 1.0 for Composite and 2.0 for RF. Quality parameter line is only informative.

Prior to these changes custom values didn't have any effect unless the preset was set to -1.0, which was a hard nut to crack for less experienced users.

nfp0 commented 7 months ago

@guestrr Oh, I see what you mean now! Thanks for the input. I need to catch up with the newer developments on your thread. :slightly_smiling_face: