casperge / RealisticSeasons-issues

1 stars 0 forks source link

Temperature update interval bug. #199

Open AxoRm opened 1 year ago

AxoRm commented 1 year ago

This configuration in the temperature.yml file has a bug. If you set the value to less than 20, then there will be changes that should not be.

  1. Death messages will be written 20/n times, where n is temperature-update-interval. I set update interval to 2 and get death messages 10 times (Test with ChatEx, Chatty, and and without any chat plugin)
  2. The player will be dealt damage more often, at the end resting on the limit per tick.(I think minecraft just can't do damage less than 1-2 times per second)
  3. Most likely, in addition to changes in the number of messages about the player's death and the amount of damage, there may be other dependencies on this parameter.

This bug is on the latest version (10.4.5), the Ssspigot 1.16.5 jar is installed on the server. Protocollib installed. No errors in the console

Why did I decide that the error was due to this plugin and not because of my fault?

  1. this happens only from deaths from temperature
  2. Changing this parameter by 20 or more corrects these errors.
  3. I tested this on an empty paper server with only your plugin.

I also test it on paper 1.16.5 and default configuration(except for this field)

casperge commented 1 year ago

Thank you for the details, will be fixed in an upcoming update