Laike-Endaril / Dynamic-Stealth

Introducing actual stealth mechanics to Minecraft
https://minecraft.curseforge.com/projects/dynamic-stealth
18 stars 2 forks source link

Crash from Level Up Reloaded Compatibility #122

Closed Ameranth closed 3 years ago

Ameranth commented 3 years ago

Hello again!

Sadly it seems that the new compatibility with Level Up is causing a crash: java.lang.IllegalArgumentException: argument type mismatch

here is the full crash log, but this seems most relevant:

at com.fantasticsource.tools.ReflectionTool.invoke(ReflectionTool.java:205)
at com.fantasticsource.dynamicstealth.compat.CompatLevelUp2.stealthLevelVisMultiplier(CompatLevelUp2.java:32)
at com.fantasticsource.dynamicstealth.server.senses.sight.Sight.visualStealthLevelInternal(Sight.java:404)
at com.fantasticsource.dynamicstealth.server.senses.sight.Sight.visualStealthLevel(Sight.java:168)
at com.fantasticsource.dynamicstealth.server.senses.sight.Sight.canSee(Sight.java:121)
at com.fantasticsource.dynamicstealth.server.ai.edited.AIWatchClosestEdit.func_75250_a(AIWatchClosestEdit.java:85)

Perhaps the Level Up stealth level is stored as a float?

Laike-Endaril commented 3 years ago

This should not happen if you use the latest version of Level Up Reloaded (1.5.8) You're currently on an older version (1.1.23)

Ameranth commented 3 years ago

Ah, fair enough; I had forgotten about not updating Level Up, as it has some questionable feature differences between versions.

Thank you for the reply!

Laike-Endaril commented 3 years ago

Hmm...well I'd suggest trying it out at least (the new version, that is). Tell me how it works out, and your thoughts on the feature differences

Laike-Endaril commented 3 years ago

Ie. if there's a good enough reason to, I'll try adding support for a previous version. But give the new one a good try first so I don't do it for no reason :P

Ameranth commented 3 years ago

No no don't worry about that, I'm happy giving it a try to have your mod working :) I'm pretty sure Level Up can be altered to work more-or-less how I want anyway.

Laike-Endaril commented 3 years ago

Alright, well in that case I'll close this for now, but if anything else comes up, feel free to reply here and I'll see it (and re-open as necessary)