Galacticraft frozen fuel problem #3534

Closed
Aragrohl opened this Issue Apr 30, 2017 · 18 comments

Comments

Projects
None yet
5 participants
@Aragrohl

This happened when using the 78 Beta galacticraft build and the 7.99.1 alpha buildcraft build, here's the log.
crash-2017-04-30_19.56.52-client.txt

@AEnterprise

This comment has been minimized.

Show comment
Hide comment
@AEnterprise

AEnterprise Apr 30, 2017

Member

looks like the BC fluidrenderer is choking on the galacticraft fuel, might be an issue on their side but the code should probably be made more robust so it doesn't completely crash

Member

AEnterprise commented Apr 30, 2017

looks like the BC fluidrenderer is choking on the galacticraft fuel, might be an issue on their side but the code should probably be made more robust so it doesn't completely crash

@Aragrohl

This comment has been minimized.

Show comment
Hide comment
@Aragrohl

Aragrohl Apr 30, 2017

Ok I'll post on their github then.

Aragrohl commented Apr 30, 2017

Ok I'll post on their github then.

@AEnterprise

This comment has been minimized.

Show comment
Hide comment
@AEnterprise

AEnterprise Apr 30, 2017

Member

looking more at it BC is purposely crashing atm when it fails to map a texture, i'm not exactly sure if it's a BC issue or that something is wrong with that fluid

@AlexIIL are you sure this is the route we want to take in this case? wouldn't it be better to use the missing texture texture instead so atleast the game can load? that texture should be indication enough that something needs fixing without breaking things on launch

Member

AEnterprise commented Apr 30, 2017

looking more at it BC is purposely crashing atm when it fails to map a texture, i'm not exactly sure if it's a BC issue or that something is wrong with that fluid

@AlexIIL are you sure this is the route we want to take in this case? wouldn't it be better to use the missing texture texture instead so atleast the game can load? that texture should be indication enough that something needs fixing without breaking things on launch

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL Apr 30, 2017

Member

Its alpha. Its crashing so that I can pick up where code doesn't work rather than just logging an error as then they wouldn't get reported.

Member

AlexIIL commented Apr 30, 2017

Its alpha. Its crashing so that I can pick up where code doesn't work rather than just logging an error as then they wouldn't get reported.

@dj3520 dj3520 referenced this issue in micdoodle8/Galacticraft Apr 30, 2017

Closed

Buildcraft frozen fuel problem #2660

@radfast

This comment has been minimized.

Show comment
Hide comment
@radfast

radfast Apr 30, 2017

thanks guys, just confirming this is not a Galacticraft issue :) (it was reported to us as well)

@AlexIIL anything we can do to help on Galacticraft's side, please comment on micdoodle8/Galacticraft#2660 - I can also give you my Skype privately so you can contact me directly if you need it

radfast commented Apr 30, 2017

thanks guys, just confirming this is not a Galacticraft issue :) (it was reported to us as well)

@AlexIIL anything we can do to help on Galacticraft's side, please comment on micdoodle8/Galacticraft#2660 - I can also give you my Skype privately so you can contact me directly if you need it

@AlexIIL AlexIIL self-assigned this Apr 30, 2017

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL Apr 30, 2017

Member

@radfast I haven’t looked into it yet but it is almost certainly a BC logic bug. I'll post more details when I've looked into it though.

Member

AlexIIL commented Apr 30, 2017

@radfast I haven’t looked into it yet but it is almost certainly a BC logic bug. I'll post more details when I've looked into it though.

@radfast

This comment has been minimized.

Show comment
Hide comment
@radfast

radfast Apr 30, 2017

@AlexIIL I think it's likely that what has happened here is that Galacticraft also registers a fluid with the same name fuel and as you know, the Forge fluid system picks one of the two mods registering the fluid as the "default" fluid for that name.

I'm actually not sure how Forge picks between the mods, but I guess in this case it picked us first ... ;)

If there's some parameter I can change when Buildcraft is present, to tell it to give your fluid priority, I don't mind doing that. Obviously we want full compatibility between our two mods. Galacticraft is built to be resilient if another mod registered fuel first. Especially with Buildcraft we have always given priority to Buildcraft's oil and fuel in pre 1.11 versions.

radfast commented Apr 30, 2017

@AlexIIL I think it's likely that what has happened here is that Galacticraft also registers a fluid with the same name fuel and as you know, the Forge fluid system picks one of the two mods registering the fluid as the "default" fluid for that name.

I'm actually not sure how Forge picks between the mods, but I guess in this case it picked us first ... ;)

If there's some parameter I can change when Buildcraft is present, to tell it to give your fluid priority, I don't mind doing that. Obviously we want full compatibility between our two mods. Galacticraft is built to be resilient if another mod registered fuel first. Especially with Buildcraft we have always given priority to Buildcraft's oil and fuel in pre 1.11 versions.

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL Apr 30, 2017

Member

nonono, its a bug in buildcraft's texture duplication + expansion code. Basically, for every fluid's still sprite it creates a new sprite called "buildcraftlib:fluid_" + srcLocation.toString().replace(':', '_') + "_convert_frozen", and if it fails to stitch this new sprite to the map (as no other mod should use the buildcraftlib namespace) then it crashes. I guess what's happening here is they you have two fluids that both use the same sprite (and so I'll tweak the impl to not recalculate a new sprite if its already done it)

Member

AlexIIL commented Apr 30, 2017

nonono, its a bug in buildcraft's texture duplication + expansion code. Basically, for every fluid's still sprite it creates a new sprite called "buildcraftlib:fluid_" + srcLocation.toString().replace(':', '_') + "_convert_frozen", and if it fails to stitch this new sprite to the map (as no other mod should use the buildcraftlib namespace) then it crashes. I guess what's happening here is they you have two fluids that both use the same sprite (and so I'll tweak the impl to not recalculate a new sprite if its already done it)

@radfast

This comment has been minimized.

Show comment
Hide comment
@radfast

radfast Apr 30, 2017

frozen fluids, sounds like fun.

We have a Liquid Nitrogen Canister item. If the player activates the item on a water block, it turns it into Ice. Maybe I can extend this to freeze all your fluids too :)

radfast commented Apr 30, 2017

frozen fluids, sounds like fun.

We have a Liquid Nitrogen Canister item. If the player activates the item on a water block, it turns it into Ice. Maybe I can extend this to freeze all your fluids too :)

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL Apr 30, 2017

Member

Its used for making the flow effect in fluid pipes - it doesn't look quite right if we use the still image directly, so we double the size of the first frame.

Member

AlexIIL commented Apr 30, 2017

Its used for making the flow effect in fluid pipes - it doesn't look quite right if we use the still image directly, so we double the size of the first frame.

@radfast

This comment has been minimized.

Show comment
Hide comment
@radfast

radfast Apr 30, 2017

I'll leave you to it, you're welcome to get in touch if any future compatibility issues arise.

radfast commented Apr 30, 2017

I'll leave you to it, you're welcome to get in touch if any future compatibility issues arise.

@AlexIIL AlexIIL closed this in ade5f9b Apr 30, 2017

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL Apr 30, 2017

Member

@radfast Sure, thanks for being friendly about this :)

@Aragrohl In theory commit ade5f9b should have fixed it, so please can you either build + test this yourself or wait until 7.99.2 and retest it to make sure its fixed :)

Member

AlexIIL commented Apr 30, 2017

@radfast Sure, thanks for being friendly about this :)

@Aragrohl In theory commit ade5f9b should have fixed it, so please can you either build + test this yourself or wait until 7.99.2 and retest it to make sure its fixed :)

@AEnterprise

This comment has been minimized.

Show comment
Hide comment
@AEnterprise

AEnterprise May 1, 2017

Member

@Aragrohl you can grab build 164 from the jenkins to test with

http://vps1.oldguys.eu/jenkins/job/BuildCraft-8.0.x/

Member

AEnterprise commented May 1, 2017

@Aragrohl you can grab build 164 from the jenkins to test with

http://vps1.oldguys.eu/jenkins/job/BuildCraft-8.0.x/

@Aragrohl

This comment has been minimized.

Show comment
Hide comment
@Aragrohl

Aragrohl May 1, 2017

@AEnterprise @AlexIIL @radfast Sorry for the bother, I might have been a bit hasty posting it to the Galacticraft github and yes it works with build 164 and build 78 of Galacticraft thanks guys.

Aragrohl commented May 1, 2017

@AEnterprise @AlexIIL @radfast Sorry for the bother, I might have been a bit hasty posting it to the Galacticraft github and yes it works with build 164 and build 78 of Galacticraft thanks guys.

@AlexIIL

This comment has been minimized.

Show comment
Hide comment
@AlexIIL

AlexIIL May 1, 2017

Member

@Aragrohl The error message isn't that helpful atm, it should really of said the BC texture that failed to overwrite rather than the Galacticraft one, but thanks for verifying :)

Member

AlexIIL commented May 1, 2017

@Aragrohl The error message isn't that helpful atm, it should really of said the BC texture that failed to overwrite rather than the Galacticraft one, but thanks for verifying :)

@LuigiHutch LuigiHutch referenced this issue in SlimeKnights/TinkersConstruct May 1, 2017

Closed

Tinkers Conflict with Buildcraft #2881

@Hypsellis

This comment has been minimized.

Show comment
Hide comment
@Hypsellis

Hypsellis May 2, 2017

I get the same crash with lightningcraft https://pastebin.com/ZNcw6CPC

Hypsellis commented May 2, 2017

I get the same crash with lightningcraft https://pastebin.com/ZNcw6CPC

@AEnterprise

This comment has been minimized.

Show comment
Hide comment
@AEnterprise

AEnterprise May 2, 2017

Member

@Hypsellis yes that is the same crash and it has been fixed, you can eighter use that jenkins builld or wait for the next release

Member

AEnterprise commented May 2, 2017

@Hypsellis yes that is the same crash and it has been fixed, you can eighter use that jenkins builld or wait for the next release

@Hypsellis

This comment has been minimized.

Show comment
Hide comment
@Hypsellis

Hypsellis May 2, 2017

Nah it's fine I will wait the next release. ;)

Nah it's fine I will wait the next release. ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment