RelativityMC / VMP-fabric

A Fabric mod designed to improve server performance at high playercounts.
MIT License
136 stars 8 forks source link

VMP and betterfpsdistance conflict #53

Open AlexanderwithaA opened 6 months ago

AlexanderwithaA commented 6 months ago

Describe the bug vmp causes betterfpsdistance to not work correctly, betterfpsdistance changes minecraft to render in a circle since fog covers the corners, having vmp stops that.

To Reproduce

  1. install the fabric versions of vmp and betterfpsdistance along with necessary dependencies.
  2. create a new world and fly up high to so you can see where the corners of the rendered area are/should be. (a mod to disable fog might be useful to see this)

Expected behavior the rendered area is circular.

Screenshots betterfpsdistance only 2023-12-26_22 58 47 both mods 2023-12-26_23 07 50

Runtime info (please complete the following information):

Crash reports / logs https://mclo.gs/ZZ3bMKT

Other mods https://pastebin.com/hstjqDpU

Checklist

Additional context I'm using vulkanmod instead of sodium

ishland commented 6 months ago

Working as intended.

VMP replaces the chunk sending to make it cost less to track sent chunks, and the square chunk sending behavior is intentional.

You have already disabled the fog, chunks behind the fog should still render.

ishland commented 6 months ago

Oops didn't mean to close this

AlexanderwithaA commented 6 months ago

I was just looking at this again because vmp was making my performance worse, mojang made chunk rendering cylindrical in snapshot 21w37a, could vmp reflect this change? https://minecraft.wiki/w/Java_Edition_21w37a#General_2

ishland commented 6 months ago

I was just looking at this again because vmp was making my performance worse, mojang made chunk rendering cylindrical in snapshot 21w37a, could vmp reflect this change? https://minecraft.wiki/w/Java_Edition_21w37a#General_2

Chunk rendering should already be cylindrical if both fog and fog occlusion is enabled. And as I mentioned above, the revert to square chunk sending behavior is intentional in VMP. Chunk loading in vanilla is also never cylindrical even after 21w37a.

AlexanderwithaA commented 6 months ago

I'm talking about rendering not loading, what I'm concerned about is that the rendering with vmp is square while in vanilla it's cylindrical. Top image is 1.18.2 vanilla, middle is 1.18.2 with vmp, and bottom is 1.20.2 vanilla. So is it intentional that vmp renders chunks in a square?

2024-01-01_16 15 26 2024-01-01_16 06 05 2024-01-01_16 16 10

ishland commented 6 months ago

It is intentional.

There are a few reasons for this: