Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

Starlight Forge 1.0.0-RC + Create 0.3.1 conflict #45

Closed
glarmer opened this issue Apr 3, 2021 · 7 comments · Fixed by #111
Closed

Starlight Forge 1.0.0-RC + Create 0.3.1 conflict #45

glarmer opened this issue Apr 3, 2021 · 7 comments · Fixed by #111
Labels
bug Something isn't working forge Forge version of the mod mod-conflict Conflict with another mod

Comments

@glarmer
Copy link

glarmer commented Apr 3, 2021

https://pastebin.com/Bvn5jx39

https://pastebin.com/cKbK8UHp

https://pastebin.com/eTbphx1i

Different crashes each time, I'm not sure which mod is at fault out of the two so posting on both.

It seemed to happen regardless of if I had creates "experimentalRendering" option set to on or off.

@Spottedleaf Spottedleaf added bug Something isn't working forge Forge version of the mod mod-conflict Conflict with another mod labels Apr 3, 2021
@Jozufozu
Copy link

Jozufozu commented Apr 3, 2021

@Spottedleaf Create puts its contraptions into a fake world and runs the lighting engine in it, so it'd be great to be able to take advantage of all the optimization work you've done here. This should be fixable on Create's end.

@Spottedleaf
Copy link
Member

It appears the chunks the world is returning is returning chunks that return null for chunk status. Typically this value should never be null.

Starlight generally expects Vanilla chunk data when it's given an IChunk. For example, I expect the chunk to have a status (I don't care about chunk generation statuses though), and I expect the block data inside the chunk to be stored in the ChunkSection[]. I access the palette of the ChunkSection directly. I also expect the World object to give me chunks, but I rely on the chunk status and the hasLight() function to determine whether light should be propagated in chunks. I also expect chunks to implement ExtendedChunk and provide accessors to the light data and emptiness maps.

I also expect worlds to implement ExtendedWorld and the getCustomLightHandler method. It can return null if no custom light is used, but be aware this "api" will be dropped in 1.17 entirely so don't get comfortable using it.

So provided the above (maybe more, I don't know everything off the top of my head) is maintained it should work fine.

@NielsPilgaard
Copy link

It appears the chunks the world is returning is returning chunks that return null for chunk status. Typically this value should never be null.

I believe this issue has been fixed in Create 0.3.2 and up :)

@GordonHollow
Copy link

crash-2021-08-27_14.05.56-client.txt
Get this crashlog witch Create 0.3.2d, so it doesn't fixed(

@jakeb072001
Copy link

crash-2021-08-27_14.05.56-client.txt
Get this crashlog witch Create 0.3.2d, so it doesn't fixed(

Yeah I get the same thing, from what I can tell its fixed in starlight rc3 but that's only for 1.17.
#56 has the same issue which was fixed in rc3.

@ghost ghost mentioned this issue Sep 26, 2021
@Corvus20
Copy link

crash-2021-08-27_14.05.56-client.txt
Get this crashlog witch Create 0.3.2d, so it doesn't fixed(

Yeah I get the same thing, from what I can tell its fixed in starlight rc3 but that's only for 1.17. #56 has the same issue which was fixed in rc3.

Well that sucks, I guess I will have to wait until all the other mods I like update to forge 1.17...

@Matthysse
Copy link

Matthysse commented Dec 12, 2021

There is still a create/starlight incompatibility in Minecraft 1.18.1 : https://crashy.net/hyW1At23jTboxYBvOtu6 (generated using Not Enough Crashes) (even with /flywheel backend off)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working forge Forge version of the mod mod-conflict Conflict with another mod
Projects
None yet
8 participants