Skip to content

Rudimentary fix for #48#365

Open
aphom wants to merge 9 commits intoryanhcode:mainfrom
aphom:main
Open

Rudimentary fix for #48#365
aphom wants to merge 9 commits intoryanhcode:mainfrom
aphom:main

Conversation

@aphom
Copy link
Copy Markdown

@aphom aphom commented Apr 22, 2026

Rudimentary fix for #48 | Creators-of-Aeronautics/Simulated-Project#315 | Creators-of-Aeronautics/Simulated-Project#329 | Creators-of-Aeronautics/Simulated-Project#94
Only for assembling. Fix for disassembling is already requested on Aeronautics' side.
(Creators-of-Aeronautics/Simulated-Project#518)

The blocks that are being assembled are iterated through to find blocks that break without support blocks and are all put forward in the list to assemble first, which fixed the bug.

Before fixes:
create_aeronautics_before_fix_low

After fixes in both mods:
create_aeronautics_after_fix_low

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 22, 2026

CLA assistant check
All committers have signed the CLA.

@ryanhcode ryanhcode self-assigned this Apr 22, 2026
@aphom
Copy link
Copy Markdown
Author

aphom commented Apr 22, 2026

There are still issues with these blocks listed below that allow placement of brittle blocks, as they also have the "wrench_pickup" tag, so they mess up the sorting because they are included in the list of brittle blocks.. so the dupe bug still happens with them. Not sure how to fix this as of right now without manually making a specific list of blocks that should be excluded from the brittle blocks list, which would probably not be feasible because of other addons' blocks.

  • Redstone Lamp, Target Block, Observer, Hopper, Pistons, Daylight Detector, Cart Assembler, Block of Industrial Iron, Block of Weathered Iron, Andesite Bars, Brass Bars, Copper Bars

From some testing it seems that these blocks mostly have issues while disassembling.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants