Suggestion: AE2 getItemByLabel(label) #856
Comments
|
Hmm, maybe an optional parameter to the existing methods that defines a filter (like |
|
I would like the call to reference the database component as the match should take NBT into account. Further, how can we handle multiple references to the same item? |
|
Using the db for this is a great idea. Not sure what you mean with 'multiple references to the same item'? |
|
Another way would be like the old bug was, but maybe return the itemStack that is in the Network as a separate table for the Craftables. That would be optional though. |
|
I retract the multiple references question. I thought you could set multiple priorities on like patterns to have more than one way to craft the same item. Even still, it should still be a single reference. |
Currently We can only get a List of items in the network. So when we try to maintain a specific amount of items for some items, we need to iterate over the items we want to keep stocked, then iterate over the getCraftables table and after that iterate over the getItemsInNetwork table. With 150 craftables and 2200 items in network, the worst case scenario is about 330k iterations for EACH item we want to have maintained at a specific level. This results in extremely long computation times and currently there seems to be no way to alter this in any way. So I propose to add the function getItemByLabel(label) function that returns the itemStack of the item in the network. Label would be the one that craftables[entry].getItemStack().label would be. Alternatively use the internal name of the item, f.ex: minecraft:planks. Or add in both ways.
The text was updated successfully, but these errors were encountered: