TeamMetallurgy / MetallurgyChisel

Metallurgy compatibility for Chisel
Other
4 stars 3 forks source link

Can't get to chiseled blocks that have Tinker's chisels #3

Open PureSpider opened 8 years ago

PureSpider commented 8 years ago

I can only ever select the "default" chisel blocks, never the ones added by MetallurgyChisel. For example: Tin Block, Copper Block, Bronze Block, Steel Block, Silver Block, ...

NEI is showing the chisel recipe but I can never select them from the chisel - not even if I go to the Metallurgy block first and then try from there.

AeroyrHwrdd commented 8 years ago

I have the same issue. There are newer metal blocks in Chisel and the metals available already with Chisel override the MetallurgyChisel textures. Can get a bit difficult with the ore dictionary in forge which is useful most of the time

PureSpider commented 8 years ago

Any news on this, @Glassmaker?

AeroyrHwrdd commented 8 years ago

I was wondering if there is a way, using Modtweaker, to make a script to add the afformentioned blocks as Variations to the appropriate Groups. I'm trying to understand it all but I'm not learning much from examples. I only know about addition and remova andl not how to specify a Group and then add the blocks as Variations to that Group. Anyone with a better understanding of the whole minetweaker scripting thing want to work with me on it?

AeroyrHwrdd commented 7 years ago

Okay @Glassmaker, I found the issue. ChiselGroup is without the underscore so tinblock, silverblock...etc Metallurgy chisel groups as you may know is tin_block...etc This needs changing

AeroyrHwrdd commented 7 years ago

@PureSpider I have a minetweaker script with fix

https://gist.github.com/AeroyrHwrdd/be8cff61a442cc50093f2a585940baed

PureSpider commented 7 years ago

Very nice, thank you!

To make the removal part of variations a little shorter, you can use mods.chisel.Groups.removeVariations("group"); to remove all variations in a group - according to the documentation.

AeroyrHwrdd commented 7 years ago

@PureSpider Oooh, I didn't see that. Thanks for that.

PureSpider commented 7 years ago

This is what I ended up doing and it worked for me: https://gist.github.com/PureSpider/6f9926967d40fbd3d14dcde8a5683c3a