H-uru / Plasma

Cyan Worlds's Plasma game engine
http://h-uru.github.io/Plasma/
GNU General Public License v3.0
202 stars 80 forks source link

Matching MSAA behavior from D3D #1545

Closed colincornaby closed 6 months ago

colincornaby commented 6 months ago

This will make the max AA setting functionally unavailable as 8x will be out of range. However - this will be more consistant with Windows where the max AA setting is typically interperated as 4x or 6x.

Hoikas commented 6 months ago

Does this close #1544?

colincornaby commented 6 months ago

Does this close #1544?

It depends. We probably need to decide.

This technically creates new issues. The high MSAA setting will do nothing on Metal hardware because 6x MSAA is a proprietary AMD extension that may only be available under Direct3D. (Maybe OpenGL on some platforms too?)

Also the 2x MSAA setting is unavailable on some hardware because like the Intel 5100, meaning the low MSAA setting does something.

A few changes have been proposed:

If we're fine with the following MSAA slider configurations in Metal:

...then we're ok and the issue is closed.

colincornaby commented 6 months ago

Another reason to consider keeping #1544 open is that it also affects the D3D renderer. The D3D renderer also surfaces MSAA modes that are unavailable for the hardware.

Hoikas commented 6 months ago

I'm going to go ahead and merge this because it's an objective improvement and doesn't preclude other fixes.