search

CyclopsMC / IntegratedDynamics

A Minecraft mod to take full and automated control of your appliances.
http://cyclopsmc.github.io/IntegratedDynamics/
MIT License
126 stars 61 forks source link

Support energy storage systems larger than MAX_INT #1222

Open shadow7412 opened 1 year ago

shadow7412 commented 1 year ago

Issue type:

Short description:

As pointed out by the LaserIO manual, the forge energy system supports energy storage up to 2.14 billion FE, and that mods such as Draconic Evolution and Mekanism store more energy than this buy "hacking" how forge energy works.

Integrated Dynamics also is restricted in this sense. But it would be very useful to look into how mods such as the above two side-step this limitation - and support it.

rubensworks commented 1 year ago

Thanks for the suggestion!

rubensworks commented 1 year ago

Energy in ID is already being handled mostly using long internally, it's only the Forge interface is restricted to integer. We used to have support for Tesla, which was long-based. But that went away, so there's no common long-based energy API anymore.

dylan-mulligan commented 1 month ago

I was wondering if this was fixed or at least a very simple fix now, as it appears this has been resolved in 1.20.1 CC: Tweaked latest (ATM9: TTS). Attached is my setup where I am trying to read from an induction matrix with integrated dynamics and computer craft side by side. For reference, these are the methods I used to interface CC with Mekanism: https://docs.google.com/spreadsheets/d/1sheh1BIeP_vSto881H6UtgXvxdbyJEfILM_uUW_UD7Q/edit?gid=895766350#gid=895766350 (Under Generic Mekanism Machine)

Edit: After talking to a CC expert, I think the solution could be adding the ability to read in RF values instead of FE, but that's just a theory.

image Reading incorrectly (MAX_INT): image Reading RF instead of FE (correct value): image Real numbers in FE: image