Defent / DMS_Exile

DMS For Exile
Other
85 stars 108 forks source link

DMS_MissionTimeout #55

Closed azhizhinov closed 7 years ago

azhizhinov commented 7 years ago

Hello there. I might be wrong, but "DMS_MissionTimeout" haven't used in your code at all, except in the config.sqf itself. And I have problem with the timeouts in a whole. Missions, static and dynamic, both, are will never expire with the default config.sqf. "DMS_MaxStaticMissions" is often ignored, and every DMS_fnc_selectMission interaction will generate new server thread.

http://www.exilemod.com/topic/61-dms-defents-mission-system/?page=270

Good luck.

HIMjack666 commented 7 years ago

Same problem since ARMA 1.68 update

eraser1 commented 7 years ago

DMS_MissionTimeout is used in the default missions and fn_AddMissionToMonitor*

Timeout issues should be fixed with latest update.

MaxStaticMissions will be ignored by missions designated to begin on server start.

selectMission shouldn't generate a new thread unless BIS changed something (out of my control) or you have a spawn in there.

azhizhinov commented 7 years ago

Hello there.

Sorry, man, but issues still exists.

Good luck.

azhizhinov commented 6 years ago

Hello.

Sorry, man, I've tried you last fixes without any luck. Mission timeouts do not honors, still. Your last fully working version from September 7, 2017 still in use, thanks for that. Good luck.

eraser1 commented 6 years ago

Max Static missions should be fixed now.

Keep in mind that missions will not time out if: 1) a player is nearby 2) the number of AI in the mission has changed recently.

So if you have either of those cases (and/or AI killing each other), the mission will not time out.

Also, I did recently observe missions successfully timing out, so I know that this isn't an issue with vanilla exile and DMS.

Perhaps you can enable some debugging logs and see what's going on?

azhizhinov commented 6 years ago

Hello. I've tried the new code once again with no luck. I've replaced only missions and config.sqf and missions are not expired. Your old code with this fixes is working fine.

Good luck.