Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

incompatible with Viafabricplus #615

Open
sh2kcs opened this issue Jan 19, 2025 · 25 comments
Open

incompatible with Viafabricplus #615

sh2kcs opened this issue Jan 19, 2025 · 25 comments

Comments

@sh2kcs
Copy link

sh2kcs commented Jan 19, 2025

Version Information

Latest, not given by user, but 0.14.6 at time of writing

Expected Behavior

Join server

Actual Behavior

Crash minecraft

Reproduction Steps

  1. Start minecraft
  2. Go to any server

Other Information

I already have mixin.chunk.palette=false disabled
https://pastes.dev/lzfjywRwHG - crash log

@2No2Name
Copy link
Member

Try adding the line mixin.chunk.no_locking=false to the lithium.properties file in the config folder

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Try adding the line mixin.chunk.no_locking=false to the lithium.properties file in the config folder

My config:
mixin.experimental=true
mixin.chunk.no_locking=false
mixin.chunk.palette=false

Everything still crashes

@2No2Name
Copy link
Member

Post the new crash report

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Post the new crash report

https://pastes.dev/j0nzoLKgW8

@2No2Name
Copy link
Member

Copied crash report in case the link goes down j0nzoLKgW8.txt

@2No2Name
Copy link
Member

2No2Name commented Jan 20, 2025

Hard to tell what is going on here, the crash is probably not caused by Lithium. Can you check by removing lithium entirely and reporting back?

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Copied crash report in case the link goes down j0nzoLKgW8.txt

I tried without vmp, now the crash report is:
https://pastes.dev/YSk2nS0q2H

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Hard to tell what is going on here, the crash is probably not caused by Lithium. Can you check by removing lithium entirely and reporting back?

It can be used without lithium to access the server.

@2No2Name
Copy link
Member

2No2Name commented Jan 20, 2025

Okay, can you try with lithium again and the following config file (replace the contents of lithium.properties with the following)

# Lithium config file for debugging.
# Debugging tip: If this config file works around the issue, remove one line and try again until the issue occurs again.
# Make sure to report back all your results if you opened an issue / support request.

mixin.ai=false
mixin.alloc=false
mixin.block=false
mixin.cached_hashcode=false
mixin.chunk=false
mixin.collections=false
mixin.entity=false
mixin.experimental=false
mixin.gen=false
mixin.math=false
mixin.minimal_nonvanilla=false
mixin.shapes=false
mixin.util=false
mixin.world=false

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Okay, can you try with lithium again and the following config file (replace the contents of lithium.properties with the following)

# Lithium config file for debugging.
# Debugging tip: If this config file works around the issue, remove one line and try again until the issue occurs again.
# Make sure to report back all your results if you opened an issue / support request.

mixin.ai=false
mixin.alloc=false
mixin.block=false
mixin.cached_hashcode=false
mixin.chunk=false
mixin.collections=false
mixin.entity=false
mixin.experimental=false
mixin.gen=false
mixin.math=false
mixin.minimal_nonvanilla=false
mixin.shapes=false
mixin.util=false
mixin.world=false

Image LOL

@2No2Name
Copy link
Member

2No2Name commented Jan 20, 2025

Remove mixin.entity=false from the config file. The other mod doesn't seem to expect lithium to disable options

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Remove mixin.entity=false from the config file. The other mod doesn't seem to expect lithium to disable options

Exactly the same error(

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Logs: https://pastes.dev/xxL9EYnUt6

@2No2Name
Copy link
Member

Oh sorry, also remove remove mixin.util=false from the config file

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Oh sorry, also remove remove mixin.util=false from the config file

Nice!

Image

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

https://pastes.dev/rcjaEskJ1T
No nice(((
I stood in the lobby for about 10 seconds and minecraft crashed

@2No2Name
Copy link
Member

Okay can you use the following file:

# Lithium config file for debugging.
# Debugging tip: If this config file works around the issue, remove one line and try again until the issue occurs again.
# Make sure to report back all your results if you opened an issue / support request.

mixin.ai=false
mixin.alloc=false
mixin.block=false
mixin.cached_hashcode=false
mixin.chunk=false
mixin.collections=false
mixin.experimental=false
mixin.gen=false
mixin.math=false
mixin.minimal_nonvanilla=false
mixin.shapes=false
mixin.world=false

mixin.entity.collisions.fluid=false
mixin.entity.collisions.intersection=false
mixin.entity.collisions.unpushable_cramming=false
mixin.util.accessors=false
mixin.util.block_entity_retrieval=false
mixin.util.block_tracking=false
mixin.util.chunk_status_tracking=false
mixin.util.data_storage=false
mixin.util.entity_collection_replacement=false
mixin.util.entity_movement_tracking=false
mixin.util.entity_section_position=false
mixin.util.inventory_change_listening=false
mixin.util.inventory_comparator_tracking=false
mixin.util.item_component_and_count_tracking=false
mixin.util.world_border_listener=false

@2No2Name
Copy link
Member

2No2Name commented Jan 20, 2025

If that still crashes, remove viafabricplus.
If that still crashes, remove viafabricplus and use the file from #615 (comment) again

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Thanks, it seems that this is really a viafabric problem, since I'm playing now and it doesn't crash. Should I create an issue on their github?

@2No2Name
Copy link
Member

No. Please report back what you did to here only so far

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Okay can you use the following file:

# Lithium config file for debugging.
# Debugging tip: If this config file works around the issue, remove one line and try again until the issue occurs again.
# Make sure to report back all your results if you opened an issue / support request.

mixin.ai=false
mixin.alloc=false
mixin.block=false
mixin.cached_hashcode=false
mixin.chunk=false
mixin.collections=false
mixin.experimental=false
mixin.gen=false
mixin.math=false
mixin.minimal_nonvanilla=false
mixin.shapes=false
mixin.world=false

mixin.entity.collisions.fluid=false
mixin.entity.collisions.intersection=false
mixin.entity.collisions.unpushable_cramming=false
mixin.util.accessors=false
mixin.util.block_entity_retrieval=false
mixin.util.block_tracking=false
mixin.util.chunk_status_tracking=false
mixin.util.data_storage=false
mixin.util.entity_collection_replacement=false
mixin.util.entity_movement_tracking=false
mixin.util.entity_section_position=false
mixin.util.inventory_change_listening=false
mixin.util.inventory_comparator_tracking=false
mixin.util.item_component_and_count_tracking=false
mixin.util.world_border_listener=false

I entered this config and deleted viafabricplus and the crashes stopped.

@2No2Name
Copy link
Member

2No2Name commented Jan 20, 2025

Use your old config again:
I assume it was

 mixin.experimental=true

@sh2kcs
Copy link
Author

sh2kcs commented Jan 20, 2025

Use your old config again: I assume it was

 mixin.experimental=true

I tried on the config above with viafabricplus - crash. I deleted it - no crash

@2No2Name
Copy link
Member

Okay since it works with lithium without viafabricplus, and you said earlier that removing lithium entirely fixes it too, we should report this as mod compatibility issue to viafabricplus. Thanks a lot for the many experiments you performed

@2No2Name
Copy link
Member

If you can report this to viafabricplus that would be great. Make sure to link them here

#615

@sh2kcs sh2kcs changed the title Crash minecraft on 1.21.4 incompatible with Viafabricplus Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants