notaz / pcsx_rearmed

ARM optimized PCSX fork
GNU General Public License v2.0
384 stars 207 forks source link

Scanlines: Possibility to set custom height (1px, 2px, …) + Operated in WYSIWYG full screen editor #287

Open porg opened 1 year ago

porg commented 1 year ago

Environment

Reproduction

Feature Request 1: You can set the scanline height (thickness)

Feature Request 2: Scanline settings in fullscreen WYSIWYG editor

notaz commented 7 months ago

Feature Request 1: You can set the scanline height (thickness)

This was easy to add (on the source) but it doesn't look that good. It can't be added to the destination because the hardware scaler is used and there is no software access to the resulting image (it's on it's way to the LCD already). Maybe something could be achived with the OMAP filtering coefficients, but that would require some lengthy research.

Feature Request 2: Scanline settings in fullscreen WYSIWYG editor

Too much work.

porg commented 7 months ago

Ad 1) Setting the scanlines on source rather than destination: As I understand it: Then they will only be properly scaled if you use a full integer scaling. Otherwise will fall victim to interpolation (on the boundary pixel lines at least) and this will get a blurry mess. But a few golden values will properly result in crisp results, which I will soon find out by some theoretical calculations and practical experiments. Looking forward to it.

Ad 1) OMAP filtering coefficients: How are the chances that you or other developers look into that? Should I raise a call/interest in the Pandora forums?

Ad 2) WYSIWYG editor too much effort. I accept that ofc. Out of curiosity: Where in particular is the effort in this? I had assumed that the WYSIWYG editor is written in a generic way and that my request would just be another instance of it. Using the current frame and then piping it through the filter whenever the user changes a value in the editor. But in reality this seems to be more complicated.