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

When breaking the Smeltery Controller and then place it in the same Smeltery Structure, the drain will display wrong texture #5162

Open
Cactusstudent opened this issue Oct 5, 2023 · 3 comments
Labels
1.18 Issue affects 1.18 1.19 Issue affects 1.19 Bug Issue describes unintended or broken behavior

Comments

@Cactusstudent
Copy link
Contributor

Cactusstudent commented Oct 5, 2023

Minecraft Version

1.18.2

Forge Version

40.2.9

Mantle Version

1.9.45

Tinkers' Construct Version

3.6.4.120

Describe your issue

Description

When you build a Smeltery Structure, then add some molten material into it. After doing all of these, digging the controller, then place it in the same place. in the abstract, the drain wont display any material texture. But in fact, the drain wlll display the lowest material's texture before digging the controller. (Because my English is poor, so here is some screenshots)

How to reproduce

  1. Build a Smeltery like this (with any materials, this one has molten netherite, gold and iron)
    2023-10-05_15 01 25
  2. Dig the controller
    image
  3. Place it in the same smeltery structure
    image
  4. You will found the drain display wrong texture

Here is a diff between the normal and the wrong
image

How to resolve it in a force way

Just dig it then place it, the texture will display correctly.

Other

Foundry also has this trouble.

Crash Report

With No Crash, just display wrongly

Other mods

JEI
Jade
Inventory HUD

Tried reproducing with just Tinkers?

Yes

Performance Enchancers

None of the above

Searched for known issues?

Checked pinned issues, Searched open issues, Searched closed issues, Checked the FAQ

@Cactusstudent Cactusstudent added 1.18 Issue affects 1.18 Bug Issue describes unintended or broken behavior Unreviewed Issue is new and is awaiting the team to review it labels Oct 5, 2023
@Cactusstudent Cactusstudent changed the title When breaking the Smeltery Controler and then place it in the same Smeltery Structure, the drain will shows wrong texture When breaking the Smeltery Controler and then place it in the same Smeltery Structure, the drain will display wrong texture Oct 5, 2023
@Random832
Copy link

Can you try planting a slimewood sapling and then forcing it to grow with bonemeal? I have a theory you may be having the same issue I ran into a while ago on 1.16 in a large modpack, and if you are able to reproduce this without a large pack that would be valuable. [I had the same behavior with smelteries, and when doing this the sapling itself would change to a log but the other blocks including the other logs, leaves, and vines, would not appear and the ghost blocks would do suffocation damage]

Also check other situations like whether the controller or drains invalidate when breaking other parts of the smeltery structure, whether this happens to drains adjacent to the controller, etc.

Also please check for error messages in the log. Even if there's no crash report there may be useful information in the logs.

@Cactusstudent
Copy link
Contributor Author

just see it, sorry. Test it in 1.18 Forge and 1.16 Forge with Tinkers' Construct and mantle only. Didn't reproduce it. Maybe Tinkers' Construct 3 has a conflict with some mods that change the game mechanism, add some features or rewrite the growth stage. I have no idea of this, because I'm not a pro. I'm sorry for that.
And thanks for your advice!

@KnightMiner
Copy link
Member

Yeah, overall this feature is implemented in a pretty messy way right now. The hard part is that fluid changing is a function of the smeltery controller, while the drains rendering is a client side property of the drains. This means that I need to both ensure the fluid stays synced with the client and somehow trigger a render update when it changes. Code could use another pass, it is more hardcoded than I'd like though.

@KnightMiner KnightMiner added 1.19 Issue affects 1.19 and removed Unreviewed Issue is new and is awaiting the team to review it labels Jun 2, 2024
@Cactusstudent Cactusstudent changed the title When breaking the Smeltery Controler and then place it in the same Smeltery Structure, the drain will display wrong texture When breaking the Smeltery Controller and then place it in the same Smeltery Structure, the drain will display wrong texture Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.18 Issue affects 1.18 1.19 Issue affects 1.19 Bug Issue describes unintended or broken behavior
Projects
None yet
Development

No branches or pull requests

3 participants