GTNewHorizons / GT-New-Horizons-Modpack

New Modpack with Gregtech, Thaumcraft and Witchery
https://www.gtnewhorizons.com/
Other
977 stars 299 forks source link

Auto smelt Tinker's modifier not working with GregTech Ores #3781

Closed mtjohnson47 closed 4 years ago

mtjohnson47 commented 5 years ago

Which modpack version are you using?

2.0.5.1 #

If in multiplayer; On which server does this happen?

Personal Server, default configs #

What did you try to do, and what did you expect to happen?

I tried mining coal and iron ore, which can both be smelted into coal/iron. I expected it to drop coal/iron ingot, hopefully duplicated from fortune. #

What happened instead? (Attach screenshots if needed)

It dropped the ore as normal. #

What do you suggest instead/what changes do you propose?

Enable autosmelting GT ores

mtjohnson47 commented 5 years ago

It also doesn't work with small ores

Bryfer commented 5 years ago

You really don't want autosmelt on your hammer. Alot of the ores gives you precious byproducts, these byproducts can even be required to advance in progress

mtjohnson47 commented 5 years ago

Then you can silk touch them, or use a pick without autosmelt. Autosmelt is still useful when you don't want to wait on furnaces, even if you don't have it on your primary pickaxe. The questbook gives you a lava crystal, it kindof makes since to use it for autosmelt.

Dream-Master commented 5 years ago

i am not sure if this can be disabled/enabled for each type of the tincer tool. After i removed random modifier i guess it is less an issue ? And btw so far i know it wont work for gt ores in general or i am wrong ?

mtjohnson47 commented 5 years ago

Hasn't worked on any GT ore. And why would it need to be disabled at all, it's an optional thing you can add, not a default thing.

Dream-Master commented 5 years ago

sure that's why I questioned if this is an issue at all. We forked tinkers source so if anyone will add gt ore support why not.

mtjohnson47 commented 5 years ago

The issue is that it's not working, whether a bug or needing mod interaction, IDK.

Dream-Master commented 5 years ago

If you rember there was a big problem with the two mod devs in the past (2 or 3 years before)(mydio and greg) So they wont support each other. Guess thats the reason why it wont work.

mtjohnson47 commented 5 years ago

Well, there's Tinker's Gregworks, so, that's probably the spot to add compatibility

Dream-Master commented 5 years ago

that is open source too. Go ahead.

Highbeam commented 5 years ago

just make sure you don't accidently enable autosmelt on ores that need EBF/vac freezer

Dream-Master commented 5 years ago

@mtjohnson47 i cant add code to Tico so what we need to do here ?

mtjohnson47 commented 5 years ago

If there's nothing you can do, there's nothing you can do. I'm unfamiliar with how the mods work, and assumed there was probably a white or blacklist somewhere.

On Sat, Oct 27, 2018, 3:33 PM Martin Robertz notifications@github.com wrote:

@mtjohnson47 https://github.com/mtjohnson47 i cant add code to Tico so what we need to do here @mtjohnson47 https://github.com/mtjohnson47 ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/GTNewHorizons/NewHorizons/issues/3781#issuecomment-433653143, or mute the thread https://github.com/notifications/unsubscribe-auth/AE3VlHOm3nQW3Ar2ShhNfifgsPNXVeMpks5upMMzgaJpZM4X5Tz5 .

Dream-Master commented 5 years ago

@mtjohnson47 is there any in config just show it to me. If it is only code wise i cant do somethings here.

mtjohnson47 commented 5 years ago

As I said, I don't know.

Dream-Master commented 5 years ago

ok.