Rewrite Item Laser Relay system with support for Slotless Item Handlers
See original GitHub issueCyclopsMC/ColossalChests#58 shows that the lasers of this mod have performance issues when interacting with Colossal Chests.
Because of this, I have implemented a simplified variant of the IItemhandler
capability which is exposed by the Common Capabilities mod.
This is required for mods like Colossal Chests that expose a huge amount of slots, which does not scale well for item transfer when a slot-based handler is used. This is not a performance issue of either mods, but simply a direct consequence of the slot-based IItemhandler
capability.
The new ISlotlessItemHandler
capability is slot-agnostic, so it moves the responsibility for finding items when extracting and finding locations when inserting to the capability provider. This makes it possible for mods like Colossal Chest to optimize these actions by indexing its inventories.
Let me know what you think. I’d be happy to assist when implementing this capability.
More details on its usage can be found here: https://github.com/CyclopsMC/CommonCapabilitiesAPI/wiki/Slotless-Item-Handler
The latest dev build of CyclopsMC/CommonCapabilities@2ebd61c25428512ff12833244d01586c0581dd53 provides this new capability. CyclopsMC/CyclopsCore@f01eb2a711373b61dfd0e6c8407cfc49be53afa9 and CyclopsMC/ColossalChests@4bf524e110ac2ad53d95293ef59695f917f38a06 have also been updated to provide an efficient implementation of this capability. CyclopsMC/IntegratedTunnels@034c933d19054cb33d9cb1848d829fd4f8d2406b is an item transfer mod that has been updated as well to make use of this capability.
Relevant dev builds: https://dl.bintray.com/cyclopsmc/dev/org/cyclops/commoncapabilities/CommonCapabilities/1.9.4-1.2.2-74/ https://dl.bintray.com/cyclopsmc/dev/org/cyclops/cyclopscore/CyclopsCore/1.9.4-0.9.0-470/ https://dl.bintray.com/cyclopsmc/dev/org/cyclops/colossalchests/ColossalChests/1.10.2-1.4.2-141/ https://dl.bintray.com/cyclopsmc/dev/org/cyclops/integratedtunnels/IntegratedTunnels/1.10.2-1.0.3-35/
Issue Analytics
- State:
- Created 7 years ago
- Comments:19 (12 by maintainers)
Top GitHub Comments
Thanks! I will implement this once I get around to it 😃
I love “already updated” 😛 I think 1.10 is outdated and obsolete by this point 😉