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

[Ars Noveau] neoforge 1.21.1 ars nouveau and natures aura crash #8

Open
5 tasks done
GetSnekt opened this issue Oct 24, 2024 · 8 comments
Open
5 tasks done

[Ars Noveau] neoforge 1.21.1 ars nouveau and natures aura crash #8

GetSnekt opened this issue Oct 24, 2024 · 8 comments
Labels
1.21 bug Something isn't working mod-compatibility Another mod is involved

Comments

@GetSnekt
Copy link

According to your testing, does the issue occur with ONLY Immersive Portals mod installed?

No

Forge Version

21.1.72 neoforge

ImmersivePortals Version

6.0.3 neoforge 1.21.1

Latest Log

https://gist.github.com/GetSnekt/5c5d26dd33123131f62168c4060135a4

Crash Report (if applicable)

https://gist.github.com/GetSnekt/139d00df729cee7f1edef75e82c0de7f

Steps to Reproduce

  1. add IPortal
  2. add ars nouveau
  3. add natures aura

What You Expected

loading into the game

What Happened Instead

crash during startup

Additional Details

it will crash the moment you combine ars and natures aura together but not if only one of them is present.

Please Read and Confirm The Following

  • I have confirmed this bug can be replicated without the use of Optifine.
  • I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
  • I have confirmed this does not happen on the fabric version (If it does please report to here instead).
  • I have confirmed I'm not using a hybrid server (Magma, Cauldron) or a different (non official) Forge Server release.
  • I have confirmed this issue is unique and has not been reported already.
@GetSnekt GetSnekt added the bug Something isn't working label Oct 24, 2024
@Nick1st Nick1st transferred this issue from iPortalTeam/ImmersivePortalsModForForge Oct 24, 2024
@Nick1st
Copy link
Member

Nick1st commented Oct 24, 2024

I need to have a look at ars noveau to see why they mixin into the chunk sync. Does that mod have anything that allows you to see far away chunks?

@Nick1st Nick1st added 1.21 mod-compatibility Another mod is involved labels Oct 24, 2024
@Nick1st Nick1st changed the title neoforge 1.21.1 ars nouveau and natures aura crash [Ars Noveau] neoforge 1.21.1 ars nouveau and natures aura crash Oct 24, 2024
@GetSnekt
Copy link
Author

GetSnekt commented Oct 24, 2024

i don't actually know anything about that but from my knowledge it does not let you see far away chunks.
EDIT: now that i think about it maybe the item called "enchanter's eye" would do it or maybe the "scry crystal" item.

@Nick1st
Copy link
Member

Nick1st commented Oct 31, 2024

Does it happen with only Ars Noveau? Or only with both

@Camriod
Copy link

Camriod commented Dec 21, 2024

Having this exact same issue, except Nature's Aura doesn't have to be present at all.
Immersive and Ars just heavily dislike eachother in chunk loading.
I had reported this issue on the for forge repo, since the modrinth page redirected me there
not sure if that info is still useful or not

@Encharrion
Copy link

I ran into this issue, without Nature's Aura. I believe the issue is that both Ars Nouveau's ChunkMapMixin.java and Immersive Portals MixinChunkMap_C.java attempt to modify onChunkReadyToSend. The Ars Nouveau mixin appears to allow chunks near a camera to be sent to the player viewing the camera, according to the comments. I'm guessing Ars Nouveau has a scrying spell that wouldn't really function without this. Not knowing anything about mixins, I wonder if both mixins being on the same priority of 1100 might be part of the issue?

@Alexthw46
Copy link

Setting Ars mixin priority higher allow to at least load together with most functionalities working.

You should refrain from using Overwrite in Mixins and use Inject or MEx (now included by default in Neo) Wrappers as Redirect to avoid mixin clashing like crazy

@Nick1st
Copy link
Member

Nick1st commented Jan 14, 2025

This mixin is an overwrite, as we want to crash rather than causing a lot of really hard to detect issues that can cause crashes and corruption later on.

@Nick1st
Copy link
Member

Nick1st commented Jan 14, 2025

I'm aware that this is not a great solution, but this needs to be fixed by adding explicit compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.21 bug Something isn't working mod-compatibility Another mod is involved
Projects
None yet
Development

No branches or pull requests

5 participants