search

openplanet-nl / issues

Issue tracker for Openplanet.
10 stars 0 forks source link

Crash on upgrade of match simulator (0.1.3 to 0.1.5) #140

Open XertroV opened 2 years ago

XertroV commented 2 years ago

image

From discord:

XertroV: with 1.24.2 I still get this trying to update match simulator from 0.1.3 to 0.1.5 (obvs not a big deal -- i don't know if i can remove it and just reinstall to fix; I'm just keeping it around atm to check if it keeps happening or not)

There are some prior msgs on discord about it too, but I don't have links on hand.

Currently I'm trying to avoid working around the issue so that it's available for debug if necessary (nothing in the logs for this sort of crash last time I checked)

codecat commented 2 years ago

MatchSimulator and PlayerState both have a (differing) AR_Data.as, meaning you have 2 different types of shared class sTMData, which probably confuses the hell out of Angelscript.

image

AR_Data.as is a shared export in PlayerState so you probably don't need your own copy of it.

codecat commented 2 years ago

I'll have to report an Angelscript bug for this failed assertion at some point though, so I'm gonna leave this issue open for now.

XertroV commented 2 years ago

I'll have to report an Angelscript bug for this failed assertion at some point though, so I'm gonna leave this issue open for now.

Are all assertion failures angelscript bugs? I have had a number of them in different circumstances -- e.g., one somewhat common one is something to do with not marking something a reference.

codecat commented 2 years ago

Depends, feel free to make GitHub issue for them here and I'll check them out.