SirEdvin / UnlimitedPeripheralWorks

Extreme compact CC:R and CC:T mod that tries to make everything, that even possible a peripheral
MIT License
9 stars 3 forks source link

AE2 API and peripheral integration #7

Closed mblaze closed 1 year ago

mblaze commented 1 year ago

I mainly use this integration to schedule some crafting depending on the number of resources available in my ME network. To do this well I need information about running crafting jobs. I find the current API to be either hard to use or lacking. I'd like you to consider three possible approaches to improving the current solution:

IMO the third option has many advantages. It would allow users of that mod to easily transfer their programs, it is a more mature design and while the API is not perfect it has much more capabilities then the current one in the UPW. Improving and extending the current API would be the most conservative option and would allow us to avoid breaking changes. This also means that we are stuck with the choices that were already made, whether they were good or not. Designing a new API is hard but with some careful consideration it might give a better user experience. Of course a hybrid approach is also possible.

Please consider the options and write back what do you think about it. I'll be glad to help with the implementation.

SirEdvin commented 1 year ago

tbh, I have no strict opinion about this, except reusing AdvancedPeripheral API, mostly because of exportItem/exportItemToPeripheral approach.

However, idea to implement the rest of kit, especially item as a table seems like a nice idea. I have plans for expanding job managing logic in future releases in any case.

SirEdvin commented 1 year ago

So, new method getActiveCraftings will be introduced to close this gap. With this, I guess it should have at least feature-parity with AP API for now. If this is not true, please, reopen this issue.