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

Don't ignore Tinker's Construct blocks in ore registration. #3709

Merged
merged 8 commits into from
Jan 3, 2025

Conversation

YannickMG
Copy link
Contributor

This lets them work with the Type Filter machines, and it shouldn't cause unwanted unification.

Incidentally this also works for the various gravel ores which would also get picked up by the type filter. This would be appropriate since they have the same macerator recipes as their regular ore counterparts.

The only side effect I noticed in the full pack is one additional duplicated compressor recipe for metal blocks of TiCo types like Cobalt, so they have 3 copies of the exact same recipe instead of 2. I'm given to understand this problem is to be handled by switching Materials system altogether and so it would be safe to ignore for the context of this PR.

Would fix GTNewHorizons/GT-New-Horizons-Modpack#15990

This lets them work with the Type Filter machines, and it shouldn't cause unwanted unification or recipe creation.
@YannickMG YannickMG added the bug fix Fix a bug. Please link it in the PR. label Dec 25, 2024
@Dream-Master Dream-Master requested a review from a team December 26, 2024 02:04
@serenibyss serenibyss added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 29, 2024
@chochem
Copy link
Member

chochem commented Jan 3, 2025

for the gravel ore: doesnt this create duplicate recipes? and different ones most likely as the current ones dont have byproducts? (looking at iron as an example right now)

Copy link
Member

@chochem chochem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok I did some testing and yea that is indeed the case:
image
image

I dont care too much about the byproduct existing, but you should at least remove the costum recipes in nhcoremod.

@YannickMG YannickMG requested a review from chochem January 3, 2025 18:02
Copy link
Member

@chochem chochem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@Dream-Master Dream-Master merged commit f2376d8 into master Jan 3, 2025
5 checks passed
@Dream-Master Dream-Master deleted the type-filter-tico-ores branch January 3, 2025 22:00
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fix a bug. Please link it in the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tinkers Construct Cobalt & Ardite Ores do not get type filtered properly.
4 participants