Minecraftforge: [Suggestion] Add global weather API

Created on 18 Jan 2019 · 7Comments · Source: MinecraftForge/MinecraftForge

many mods have different wind turbine systems.
and they all calculate the wind in different ways.

Add global weather API

Weather heat, wind, humidity.
Every blocks has own heat.
Every fluid has own heat and its changing over time depending weather heat.

Stale

Source

anar4732

Most helpful comment

In my opinion this falls under "content" and should be a mod rather than a built-in in Forge. However hooks to modify heat values would be good if not already present.

SkyHawkB on 18 Jan 2019

👍7

All 7 comments

In my opinion this falls under "content" and should be a mod rather than a built-in in Forge. However hooks to modify heat values would be good if not already present.

SkyHawkB on 18 Jan 2019

👍7

Well it could be said the same when Forge added an energy api while RF existed already. Add a weather api wouldn't be a bad idea. It would enable all modders to use the same weather api, ease them to code their mod, and create a retro compability between all mods. Modders will have in a way or another to rewrite all their mods in 1.13. So it is surely the best and only time for adding such content in Forge. I would extend the idea by addind a temperature, humidity, thirst, and tiredness API.

Hypsellis on 2 Feb 2019

It would enable all modders to use the same weather api, ease them to code their mod, and create a retro compability between all mods.

None of that requires the API being in Forge. Modders can and should create their own APIs to make similar systems in different mods compatible. Every new hook or API added to Forge has to be maintained, so Forge is very picky. The only APIs that go in Forge are those that are very commonly used, like the fluid & energy APIs.

a temperature, humidity, thirst, and tiredness API

These aren't useful to many mods, and the mods that do add these features usually shouldn't be compatible with other mods adding the same features. If compatibility is necessary, it is perfectly possible for the respective mod teams to create a common API.

Daomephsta on 2 Feb 2019

👍3

Temperature could be viable, only because torches melt ice. Light shouldn't melt ice, heat should.
However, none of the other weather has any effect in vanilla

Alpvax on 2 Feb 2019

👍1

Well it could be said the same when Forge added an energy api while RF existed already.

Wrong! The reason Forge added its energy API was because there were a large amount of mods all using different energy APIs and it was decided that one standard API would be better than having to convert between 3-4 different APIs (if you don't believe this, look at the conversion table). However, very few mods have exising weather APIs and very few modders would benefit from a weather API being added to Forge.

I would extend the idea by addind [sic] a temperature, humidity, thirst, and tiredness API.

The former two somewhat exist in Minecraft itself as biome properties but could be improved. The latter two are, as mentioned earlier, not worthwhile to implement in Forge because so few mods would use them.

Edit: just realized Daomephsta said this right above me but I'll leave this here just because deleting it is pointless. And as they said, the Forge team is very picky as to what to add.

SkyHawkB on 2 Feb 2019

👍2

This issue has been automatically marked as stale because it has not had activity in a long time. If this issue is still relevant and should remain open, please reply with a short explanation (e.g. "I have checked the code and this issue is still relevant because ___." or "Here's a screenshot of this issue on the latest version"). Thank you for your contributions!