Skip to content
This repository has been archived by the owner on Aug 18, 2024. It is now read-only.

Dank Dock + Ezpass pipes not filling dank properly #7

Closed
Diamond-anvil opened this issue Oct 9, 2020 · 17 comments
Closed

Dank Dock + Ezpass pipes not filling dank properly #7

Diamond-anvil opened this issue Oct 9, 2020 · 17 comments

Comments

@Diamond-anvil
Copy link

Playing All of Fabric 3 v2.0.5 when using the dank dock + dank as bulk storage on a mob farm. The dank is filled based on stack sizes of 64 not the dank's slot storage capacity. As items come into the dock they are distributed to the next available slot in the dank until the stack size reaches 64 at which point the next item is placed in the next available empty dank slot. To mitigate this issue I have to resort the dank to distribute the items into there proper dank slots. When all empty/available slots are filled to stack size 64 the dock stops accepting items.

@Tfarcenim
Copy link
Owner

Does it work with hoppers?

@Diamond-anvil
Copy link
Author

No, I get the same result when feeding the dock via hoppers. The first slot in the dank fills to 64 items then moves on to the next available slot.

@Tfarcenim
Copy link
Owner

Hmm, can I see the debug/latest.log? I suspect an interfering mixin.

@Diamond-anvil
Copy link
Author

Hot off the presses.

latest.log
debug.log

@Tfarcenim
Copy link
Owner

Damn still can't see anything, no way to debug this without removing mods.

@minedak
Copy link

minedak commented Nov 12, 2020

Wanted to make a similar issue, but Diamond-anvil beat me to it. Maybe the findings of some minor experimentations might be useful to add, though.

I've started a relatively clean (modwise) new world with only a single hopper pointing into a dock with a dank3 in it, and loads of mc:stone in the hopper. As long as I add some light-weight mods, things keep working as they should. Once I add either of the following mods, dank slots will only be filled to 64 (after which a new slot is used for the next 64, and so on):

  • Modern Industrialization
  • Astromine: Complete
  • Industrial Revolution

I've tried all three of them separately, and made sure that after disabling them normal functionality was restored (didn't disable dependencies like patchouli and kotlin, but they didn't seem to affect dank-filling to beyond 64). So it doesn't appear to be a single mod breaking things, but maybe some library shared by all of the above mods?

@Tfarcenim
Copy link
Owner

Interesting, so one of those mods or their dependents does something to the hopper as well. It would be wise to link them this issue.

@flier268
Copy link

flier268 commented Dec 1, 2020

Because this mod use officialMojangMappings but yarn?

@Tfarcenim
Copy link
Owner

nope, mappings have nothing to do with what names were used with it.

@flier268
Copy link

flier268 commented Dec 2, 2020

Some lib override the class, like this mode
This is Astromine's lib

@flier268
Copy link

flier268 commented Dec 9, 2020

not support AE2

@MissPotato
Copy link

MissPotato commented Dec 15, 2020

Sorting the dank dock works fine, maybe as a band-aid fix add an invisible slot that when filled auto-sorts the dank? It obviously doesn't fix the issue, but it makes the Dock insertions usable.

@Gerdathlete
Copy link

Gerdathlete commented Dec 17, 2020

This is an issue with the Fabric Dank Storage mod, not the pipes/input method

I'm having this issue regardless of what mod inserts items into the docked Dank. AE has the issue, Modern industrialization pipes have the issue, minecraft hoppers have the issue, and more. For some reason the inventory is treated like a normal chest so items only stack to 64 when inputted via pipes/hoppers, and the dank will no longer accept items once all of its slots contain 64 items. Additionally, when sorting the dank, the stacks do not combine, so it does not provide a temporary solution.

@Tfarcenim
Copy link
Owner

No, this is what happens when there's no item api in Fabric like there is in Forge which is an issue with Fabric itself. Mainline Dank Storage doesn't have this issue for that reason.

For the record, hoppers do work when installed with only vanilla because they're patched to do so. But other mods patch the hopper again which breaks it.

@AlexIIL
Copy link

AlexIIL commented Dec 17, 2020

It looks like this is a LibBlockAttributes interaction issue: (AlexIIL/LibBlockAttributes#39).

Essentially the 64 stack limit comes from ItemStack.getMaxCount(), and not LBA itself, however LBA currently uses that limit when inserting items into inventories. (Although for the vanilla hopper case that should be easier to fix: LBA just needs to know to not use LBA's insertion methods for Dank's blocks - I'm not sure exactly how to find this out though).

That wouldn't help for LBA-dependant blocks though, since they will adhere to the maximum stack count anyway. A quick fix would be to expose an LBA FixedItemInv with a custom insertStack method, either as an only-loaded-if-lba-is-present class, or just hard dep LBA (which I assume you'd rather not do).

@AlexIIL
Copy link

AlexIIL commented Dec 17, 2020

All mods that use LBA (plus hoppers) should be fixed if you (manually) update LibBlockAttributes to 0.8.5. That will fix AE2, Modern Industrialization, Astromine, and Industrial Revolution. I'm not sure about ezpass though.

@Tfarcenim
Copy link
Owner

3.3 uses the new item api

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants