Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subnet contents not visible through storage busses #17950

Open
3 tasks done
Kyramesh opened this issue Nov 12, 2024 · 2 comments · May be fixed by GTNewHorizons/Applied-Energistics-2-Unofficial#612
Open
3 tasks done
Labels
Bug: Minor Mod: AE2 Status: Ready for Developer Issue ready for a developer to pick up and implement Type: QoL

Comments

@Kyramesh
Copy link

Your GTNH Discord Username

Kyramesh

Your Pack Version

2.7 Beta 3

Your Server

SP

Java Version

Java 21

Type of Server

None

Your Expectation

Making contents of an AE net available through Interface -> filtered Storage bus connections.

The Reality

Content of net A is inexplicably not visible in net B (in terminal and for autocrafting). Examples:
https://discord.com/channels/181078474394566657/522098956491030558/1305298344259883159
Stocking Bus seems to be able to access contents even though a terminal can't.

https://discord.com/channels/181078474394566657/522098956491030558/1305901151102963743
Having storabge busses "importing" and "exporting" contents, especially if mutltiple nets are involed, seems to cause this, even if the storage busses are filtered in such a way that no recursive loop could form.

I think this issue then causes the following error when attempting to order crafts, which involve items from other nets:
grafik

I can't consistently cause the issue to arise. The system works fine for a while and then stops, for yet to determine reasons.

Your Proposal

Identify the issue and fix it.

Final Checklist

  • I have searched this issue tracker and there is nothing similar already. Posting on a closed issue saying the bug still exists will prompt us to investigate and reopen it once we confirm your report.
  • I can reproduce this problem consistently by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
  • I have asked other people and they confirm they also have this problem by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
@Kyramesh Kyramesh added Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed labels Nov 12, 2024
@Kyramesh
Copy link
Author

Kyramesh commented Nov 12, 2024

Examples of the filtered storage busses:
grafik
grafik
(quicklime later added here to fix the problem)

@hiroscho
Copy link

Oh hell yeah, here comes the bug report where I get a headache trying to fix my mess.

The extract filtering has two slightly different behaviors:

  1. Get all available items
  2. Get a single item

The first case is a dumb implementation that just gets all available items from the connected network and then filters them, this one is the problem.
The second one doesn't have this problem since it checks the filter first and only then checks the connected network.

The reason why the problem exists is that AE2 has recursion protection, meaning the same network will not be checked twice. In this case the first storage bus gets all available items and filters them, this is fine. However the second storage bus when trying to get all available items will notice that the network was already scanned once and skip the scan.

@RAFAEL-SOSA-UH RAFAEL-SOSA-UH added Mod: AE2 Status: Ready for Developer Issue ready for a developer to pick up and implement Type: QoL and removed Status: Triage Issue awaiting triage. Remove once this issue is processed labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Minor Mod: AE2 Status: Ready for Developer Issue ready for a developer to pick up and implement Type: QoL
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants