GTNewHorizons / GT-New-Horizons-Modpack

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

gt++ fuel rod centrifuging gives non-unified pu239 #5879

Closed chochem closed 3 years ago

chochem commented 4 years ago

Which modpack version are you using?

2.0.8.8 #

I thought the gt++ fuel rod recipes were just a work in progress or something like that. But alk tells people to use them, so here we go:

1. centrifuging is only possible for single depleted rods which practically no one uses.

  1. centrifuging depleted MOX or uranium gives a gt++ pu239 variant which is not unified with the already existing (and actually useful) pu239.

Edit: alk just wrote me somewhere else that 1 is intended.

draknyte1 commented 4 years ago

Did you not read? It’s intended. https://github.com/GTNewHorizons/NewHorizons/issues/4826

2) unification issue, not my issue.

chochem commented 4 years ago

Did you not read? It’s intended.

4826

I wrote the issue before seeing your reply obviously. I cut it down to point 2 then.

chochem commented 4 years ago

@Dream-Master why did you close this? Do you disagree that pu239 should be unified?

Dream-Master commented 4 years ago

is the recipe a gt++ recipe or a gt one @chochem

chochem commented 4 years ago

the recipe is gt++ obviously.

I checked the oredicts btw. From the little I understand, this should be the easiest place to fix this. They are: dustPlutonium for the regular plutonium 239 dust dustPlutonium239 for the gt++ plutonium 239 dust

Dream-Master commented 4 years ago

are the standard gt plutonium dust are the plutonium 239 or add alk a new isotope?

chochem commented 4 years ago

I assume the non-gt++ one is just from gt but I am just guessing there.

Here, I made you screenshots of the two:

2020-04-19_07 39 01 2020-04-19_07 38 56

leagris commented 4 years ago

They are both plutonium239. I think both GT and GT++ plutonium could have both Plutonium239 and AnyPlutonium material name and ore suffix. AnyPlutonium for all plutonium variants like isotope 241, the same way as AnyCopper or AnyIron?

Dream-Master commented 4 years ago

if this is an unification issue on gt side I'll try to fix

draknyte1 commented 4 years ago

Do NOT add anyX tags to stuff. I’ll fix this from my side, someone just needs to make me a ticket.

It’s not currently oredicted to dustPlutonium because it’s not standard PU in earlier versions of GT. refer to https://github.com/Blood-Asp/GT5-Unofficial/commit/4061f8f397d40fb3a4e363d86f47bd631a2da539

Dream-Master commented 4 years ago

@draknyte1 ok

Prometheus0000 commented 3 years ago

@draknyte1 is this fixed in a version we have? It doesn't appear so ingame

draknyte1 commented 3 years ago

Converted in chem plant. Fixed in 9.0 afaik.

Prometheus0000 commented 3 years ago

OK thanks