Configuring one Shader for all emulators on startup (no hotkeys desired)

davej

The shader really needs 4 or more pixels vertically for each scan line to look good with non integer scales. You could try setting up a custom screen resolution to effectively have 3.5 scaling - that gives alternate darker and lighter scan gaps lines which isn't perfect but does give an even result overall. For SNES games this would be 784 pixels tall and cropping it to 768 would mean you only lose about 5 game pixels vertically.

You are correct about the most recent released version. I did also post it this petRockBlog thread but it may not have been copied into the retropie distribution.

BuZz

You can submit PR to update the scraper at https://github.com/RetroPie/common-shaders/tree/rpi

movisman

@davej

Hi,

Ah, yes that makes sense, thanks for the advice. I think 3.5 scaling is my only option really unless I want everything to be either too overscaled or too small. 3.5 would work well on SNES, but maybe not so well on other emulators? Not really experimented with custom resolutions for each emulator, but i'm thinking that the overlay/shader combo i'm running at the moment isn't a bad compromise.

As for the version, can't believe I missed that post further down, i'm sure I read that thread. Anyhow, that's the same version as what I plucked off the libretro forums though, so is all good for me.

movisman

@BuZz

I'm not familiar with PR's, I just gave it a go but not sure if i've made a hash of the process or not. I have a feeling I have done this incorrectly. Are you able to confirm?
See here: https://github.com/gizmo98/common-shaders/pull/1
I just clicked your link above, opened a new pull request, uploaded the zip, put a description, and that was about it.

BuZz

You need to do the PR against the RetroPie repository though not gizmo98 - I really was referring to the author to submit it, but might just be easiest if you open an issue and link to the latest version and we will update it.

movisman

@BuZz

Jeez, bit embarrassing that one, just a lack of understanding on my part. I will try to remove the PR and will raise it as an issue with a link to the correct zip. Apologies.

movisman

@BuZz

Issue raised, sorry about that. Now i'm reading further about how PR's work it makes more sense now. I shouldn't have tried to raise something I didn't fully understand!

BuZz

No probs :) Thanks

jimd0586

@davej

I looked in the file and the "//" prior to define were not there. So curvature should be working... so yeah. I have no clue what's up. I swear it worked the first time I copied it in.

I guess I don't really know enough to understand how to fix my problem.

jimd0586

Actually figured it out. I erased all the shaders I put in and then placed them back in. Seems to work now with curvature.