Export bus and Interface overfill insciber #547

Closed
Staegrin opened this Issue Dec 5, 2014 · 9 comments

Comments

Projects
None yet
5 participants
@Staegrin

Staegrin commented Dec 5, 2014

  1. The problem that is happening?
    I wanted to test the latest version of AE. I installed it and turned off channels in the config to make it simpler to test the basics. I stacked 5 inscribers on top of each other as I'd done in earlier versions.
    The top four got interfaces put on the back of them and the correct press was put inside. The interfaces got one pattern each for the printed silicon, printed calculation circuit, printed engineering circuit and the printed logic circuit.
    The bottom inscriber has been hooked up to three export buses. One to constantly supply the centre slot with redstone, one to fill the bottom slot with printed silicon and the last with an expansion to export the various other printed circuits.
    The problem lies with the inscriber being filled with more than one item per slot and not working because of this. Manually or with a hopper only one item per slot can be put in and the inscriber quickly does its job. When an interface or export bus is used the slots get filled with more items up to a stack and the inscriber does nothing.

  2. What was expected?
    For the inscriber to work with more than one item per slot or for the export bus and interface to only be allowed to put in one item at a time.

  3. How to reproduce the problem?
    Use an export bus to put items in the inscriber.

  4. Server or Single Player?
    single player

  5. Mod Pack using and version?
    appliedenergistics2-rv2-alpha-28.jar
    Severely modified Bevo's techpack.
    modlist:
    http://pastebin.com/f5VgPgKK (edit by @Cisien: pastebin is your friend)

  6. Crash log (Please make sure to use pastebin for the log file)
    no crash

  7. Screen shots or Pictures of the problem
    Inscriber with silicon press at the top and centre filled by interface with silicon.
    inscriber for printed silicon
    Inscriber at the bottom. Centre and bottom filled by export buses. Inscriber only works if only one of each item is present.
    inscriber for combining
    Sotal setup. Everything works if items are put in manually thanks to turning of channels the wiring doesn't matter.
    2014-12-05_01 41 14

Edit:By adding hoppers in-between the interfaces/export buses and the inscribers the system works.
2014-12-05_02 13 40

@Cisien

This comment has been minimized.

Show comment
Hide comment
@Cisien

Cisien Dec 5, 2014

Contributor

Pastebin is your friend.

Can you reproduce this with only ae2?

Contributor

Cisien commented Dec 5, 2014

Pastebin is your friend.

Can you reproduce this with only ae2?

@Staegrin

This comment has been minimized.

Show comment
Hide comment
@Staegrin

Staegrin Dec 5, 2014

I'll make an instance with only AE (appliedenergistics2-rv2-alpha-28.jar) and NEI. To test it again.

Staegrin commented Dec 5, 2014

I'll make an instance with only AE (appliedenergistics2-rv2-alpha-28.jar) and NEI. To test it again.

@Staegrin

This comment has been minimized.

Show comment
Hide comment
@Staegrin

Staegrin Dec 5, 2014

Just tested with an instance with only forge 10.13.2.1240 NEI and AE2 (appliedenergistics2-rv2-alpha-28.jar) on it.
The same problem repeats itself. The Export buses and Interfaces fill the inscribers with more than one item per slot.

Inscriber filled by interface.
ae2 only1
Insciber filled by export buses.
ae2 only2

Staegrin commented Dec 5, 2014

Just tested with an instance with only forge 10.13.2.1240 NEI and AE2 (appliedenergistics2-rv2-alpha-28.jar) on it.
The same problem repeats itself. The Export buses and Interfaces fill the inscribers with more than one item per slot.

Inscriber filled by interface.
ae2 only1
Insciber filled by export buses.
ae2 only2

@Staegrin Staegrin closed this Dec 5, 2014

@Staegrin Staegrin reopened this Dec 5, 2014

@Staegrin

This comment has been minimized.

Show comment
Hide comment
@Staegrin

Staegrin Dec 5, 2014

Sorry accidentally clicked close and comment instead of comment.

Staegrin commented Dec 5, 2014

Sorry accidentally clicked close and comment instead of comment.

@Cisien

This comment has been minimized.

Show comment
Hide comment
@Cisien

Cisien Dec 5, 2014

Contributor

@yueh related to the stack size fix with accelerators?

Contributor

Cisien commented Dec 5, 2014

@yueh related to the stack size fix with accelerators?

@Staegrin

This comment has been minimized.

Show comment
Hide comment
@Staegrin

Staegrin Dec 5, 2014

The export buses and the inscribers have accelerators. Even with them removed the same overfilling occurs.

Staegrin commented Dec 5, 2014

The export buses and the inscribers have accelerators. Even with them removed the same overfilling occurs.

@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Dec 5, 2014

Member

Related to #519. But the question is does the inscriber or bus/interface not handle it correctly.
As the inscriber does not care to report with isItemValidForSlot() if the slot is not empty.
Resulting in the bus/interface just pushing more items into it, as it does not care about the stacksize when the inventory says everything is fine, just insert.
Or both.

Member

yueh commented Dec 5, 2014

Related to #519. But the question is does the inscriber or bus/interface not handle it correctly.
As the inscriber does not care to report with isItemValidForSlot() if the slot is not empty.
Resulting in the bus/interface just pushing more items into it, as it does not care about the stacksize when the inventory says everything is fine, just insert.
Or both.

@galaxyoyo

This comment has been minimized.

Show comment
Hide comment
@galaxyoyo

galaxyoyo Dec 5, 2014

Same problem, but with channels enabled

Same problem, but with channels enabled

@thatsIch

This comment has been minimized.

Show comment
Hide comment
@thatsIch

thatsIch Dec 5, 2014

Member

this problem is unrelated to channels

Member

thatsIch commented Dec 5, 2014

this problem is unrelated to channels

yueh added a commit that referenced this issue Dec 5, 2014

Merge pull request #553 from yueh/fix-547
Fixed overstocking items with export bus/interface #547

@thatsIch thatsIch closed this Dec 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment