[ref:cargo-polishing] Cargo polishing

[LordIdra]

• Add vanilla block compatibility to cargo extractor and inserter
• Update all existing blocks to be logistics compatible
• Abstract a lot of inserter/extractor logic out into a new interactor class (it was getting a bit much with the new logic for vanilla blocks)
• Allow selecting the target inventory in extractors/inserters
• Allow setting a filter (and blacklist/whitelist)
• Adds a buffer to cargo extractor (but not inserter) - this is simpler for the filtering logic and drastically reduces the number of comparisons needed when moving items between e.g. two double chests

[balugaq]

Unable to transport Oak log into smelting slot (it can be both fuel and smelting)

Expected:

if item is fuel:
  if fuel slot if full:
    try transport to smelting slot.
  else:
    transport to fuel slot

[balugaq]

Unable to transport items into DecoratedPot

[balugaq]

QQ202612-215938.mp4

When transport items from containers, the redstone state didn't get updated.

[balugaq]

When transport into crafter, it ignored disabled slots

[balugaq]

When Cargo Extractor or Inserter was placed near the shulker box, the box couldn't be pushed by piston

[balugaq]

Unable to transport books into lectern

[LordIdra]

Unable to transport books into lectern

Intentional, not supported

Unable to transport items into DecoratedPot

Intentional, not supported

[LordIdra]

Unable to transport `Oak log` into smelting slot (`it can be both fuel and smelting`)

Expected:

if item is fuel:
  if fuel slot if full:
    try transport to smelting slot.
  else:
    transport to fuel slot

Unable to reproduce. This might be an issue with cargo routing, there are several fixes in subsequent PRs.

[LordIdra]

QQ202612-215938.mp4
When transport items from containers, the redstone state didn't get updated.

Unable to play the video but I think you're using a chiselled bookshelf there to test? Chiselled bookshelves are a bit screwed atm because of paper, not because of pylon, so not surprised that this is an issue. I have tested with other containers and redstone state updates just fine

[LordIdra]

When transport into crafter, it ignored `disabled slots`

This is a really unfortunate issue which is extremely hard to fix with the system as it's set up. I'm going to leave this and open an issue instead because I need to think on it and I'm not sure how to fix it yet

[LordIdra]

When Cargo Extractor or Inserter was placed near the shulker box, the box couldn't be pushed by piston

This is intended behaviour because of how the multiblock system works; it can't handle multiblock components being pushed around by pistons

[balugaq]

QQ202612-215938.mp4
When transport items from containers, the redstone state didn't get updated.

Unable to play the video but I think you're using a chiselled bookshelf there to test? Chiselled bookshelves are a bit screwed atm because of paper, not because of pylon, so not surprised that this is an issue. I have tested with other containers and redstone state updates just fine

Also tested hopper

Expected:

if item is fuel:
  if fuel slot if full:
    try transport to smelting slot.
  else:
    transport to fuel slot

Unable to reproduce. This might be an issue with cargo routing, there are several fixes in subsequent PRs.

This comment I changed yesterday but didn't change probably due to lag.
So when I find this bug, I didn't find the cycle button. Because the first time I see the cycle button, there's no any cycle text on it. I didn't think it has some special features. This feature is somewhat a bit hidden for me.