CyclopsMC / IntegratedTunnels

Transfer other energy, items and fluids over Integrated Dynamics networks
MIT License
20 stars 13 forks source link

Items inserted by passive interaction ignores importer's setting #308

Open AlisonHuang777 opened 1 week ago

AlisonHuang777 commented 1 week ago

Issue type:


Short description:

When inserting items with external devices (e.g. hopper), the items will always try to go to the interfaces of which the channel is set to 0 (which is default value), ignoring that of the importer. Also if round-robin is enabled it will also get ignored. Therefore I consider that items, as the title described, will omit the settings of the importer and follow the default behavior. Currently only tested with item importers/interfaces, but same issue might occur with other variation as well.

Steps to reproduce the problem:

Please watch this footage: https://youtu.be/4jYrVpU2A7c Using ATM9 0.3.2.

Expected behavior:

Items should only go to interfaces with channel 1 and the amount of them being transferred to each of the barrel should be the same (at least be close).


Versions:

Log file:

no response

rubensworks commented 1 week ago

Thanks for reporting!

rubensworks commented 4 days ago

I'm unable to reproduce your problem. Can you try reproducing the problem without any other mods installed? If so, could you share a video or a more details step-by-step guide on how to reproduce your problem?

Jack-McKalling commented 4 days ago

Did you possibly use a negative value for the Item Interface channel setting?

Or did you use a negative value for the source slot to pull from, instead? (separate question)

AlisonHuang777 commented 2 days ago

Sorry for the late update, and sorry again for that there were mistakes in the original description. Therefore I have recorded a footage to showcase the bug more clearly, link in the edited description.

Jack-McKalling commented 2 days ago

Ah. So you're using round-robin mode, but that's actually confusing the situation. Try to see if the problem happens without using round-robin, to see what really happens. That setting already deliberately ignores all priority settings according to its tooltip (see CyclopsMC/IntegratedDynamics/issues/1406).

With my own observations, it looks like with the passive interaction enabled, and the hopper pointing into the importer, items are sorted through the incorrect channel 0 as well as the correct one. With the hopper pointing elsewhere and exclusively the importer pulling form it, this will not happen.

However, interestingly, with the passive interaction setting disabled, it still happens! So there must be something wrong with the hopper interaction, maybe overriding the passive interaction setting or just ignoring the channel setting.

New repro steps;

rubensworks commented 1 day ago

Thanks for the additional information @AlisonHuang777 and @Jack-McKalling!