[QUOTE=dave j;38908] @dankcussions The crt-pi-mask was to help people create overlays but since you can get the same functionality by loading the 240p test suite with the crt-pi shader I consider it redundant. Similarly with the crt-pi-barrel shader. It does curved screens but without any extra processing so you get aliasing. The crt-pi shader can be configured to do curved screen without scanlines but with anti-aliasing (described in this post). Unless anybody is particularly keen on them, I can see no reason not to remove them.[/QUOTE]
great work! tested and works great.
my only thought is that maybe it’s better to have a separate version of the shader that has the barrel turned on, to make it an obvious option to users. would you be averse to me doing the edit to turn on the barrel, and storing that as a separate crt-pi-barrel.glslp in the retropie shader repo? maybe a crt-pi-barrel-vertical.glslp as well I suppose!
EDIT: also, as a thought - should any potential barrel/curvature version also have the multisample param uncommented, to cut down the aliasing? or is curve + multisample + scanlines too much for the pi? i tried it just with curvature + scanlines and it looked fine, albeit obvious jaggies around the edge.