CntoDev / cnto-additions

Other
0 stars 1 forks source link

Some mods are slowing down CBA XEHs #4

Open freghar opened 3 years ago

freghar commented 3 years ago
WARNING: Car does not support Extended Event Handlers! Addon: @project_opfor
WARNING: Tank does not support Extended Event Handlers! Addon: @acre2
WARNING: Air does not support Extended Event Handlers! Addon: @ace
WARNING: TH_SNOWING1 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_SNOWING2 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_SNOWING3 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_SNOWING4 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_FOG1 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_FOG2 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_FOG3 does not support Extended Event Handlers! Addon: @thirsk
WARNING: TH_SNOWSTORM1 does not support Extended Event Handlers! Addon: @thirsk
WARNING: Ryanzombieslogiceasy does not support Extended Event Handlers! Addon: @zombies_and_demons
WARNING: Ryanzombieslogicspawnfast1opfor does not support Extended Event Handlers! Addon: @zombies_and_demons
WARNING: RyanZombieCivilian_F does not support Extended Event Handlers! Addon: @zombies_and_demons
WARNING: RyanZombieB_Soldier_base_F does not support Extended Event Handlers! Addon: @zombies_and_demons
WARNING: RyanZombiePlayer1 does not support Extended Event Handlers! Addon: @zombies_and_demons
WARNING: I_E_Soldier_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Soldier_TL_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_soldier_M_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Survivor_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Medic_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Quadbike_01_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Offroad_01_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Offroad_01_covered_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Offroad_01_comms_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Van_02_vehicle_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
WARNING: I_E_Van_02_medevac_F_OCimport_01 does not support Extended Event Handlers! Addon: @jfo_park_rangers
...
WARNING: One or more children of class CAManBase do not support Extended Event Handlers. Fall back to loop.

Now, the list of related mods is probably not 100% correct.

Now, it says the issue is really only with CAManBase, but since @enhanced_movement essentially removed CBA's ability to watch EHs for Car, Tank and Air, some pretty common names people use for CBA class event handlers, I'm afraid we need to fix/report/both all of them to stop CBA from looping.

freghar commented 3 years ago

@enhanced_movement submitted as https://github.com/SceptreOfficial/Enhanced-Movement-Rework/pull/12

freghar commented 3 years ago

@JamesTheClarke Heya, do you still plan to remove @jfo_park_rangers (in the future) as was mentioned on the January MMT meeting? I don't see it mentioned anywhere aside from the meeting notes, so just confirming. Thanks!

freghar commented 3 years ago

I would like to keep this open until

in order to verify that the warnings are gone.

JamesTheClarke commented 3 years ago

Both JFO mods have been removed from our Main Repository: https://www.carpenoctem.co/forums/m/26081621/viewthread/16086660-a3sync-repository-status/post/139466358#p139466358

Please test again and close this issue if the remaining mods (Zombies and Thirsk) are not causing overtly problematic performance issues. Otherwise, we might want to start a Staff Forum discussion on removing these mods if they can't be fixed/made compatible with ACE3 damage handlers.

JamesTheClarke commented 3 years ago

@freghar can this issue be closed or are there further adjustments we should make on the server/repo?

freghar commented 3 years ago

The errors pointing out specific mods are gone, but

WARNING: One or more children of class CAManBase do not support Extended Event Handlers. Fall back to loop.

still remains.

Somebody need to (1) fidn out which mod is causing it by starting the game with some mods "unticked" and (2) figure out what exactly is CBA checking for. Then we can perhaps finally fix it.