supertuxkart / stk-code

The code base of supertuxkart
Other
4.36k stars 1.03k forks source link

Don't allow do disable objects from the track through geometry detail #5052

Closed Alayan-stk-2 closed 1 week ago

Alayan-stk-2 commented 1 month ago

Low geometry mode is meant to help performance, but it's rather inefficient at that and has some unacceptable drawbacks.

It looks horrible, but more concerning, it can affect gameplay significantly.

For example, with geometry details to the minimum, the Black Forest train cannot hit the kart. Affecting gameplay to this degree is not an acceptable tradeoff.

It doesn't make sense to remove it for 1.5, but this should not be allowed for 2.0.

Alayan-stk-2 commented 1 month ago

It appears that the geometry details setting is not always properly applied - setting it to the lowest value and starting Black Forest, it sometimes still has trees, which is incorrect.

The issue mentioned above only happens when the geometry detail culling properly happens, but why it doesn't always happen deserves an investigation too.