Wrong default value for "AllowParallelAIQueues" tag

Bug #910146 reported by FS-21
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ares
Fix Released
Low
Renegade

Bug Description

Ares version: 0.1.1169
S.O.: Windows 7 pro (64 bits)

About "Multiple AI Factories Clone Units" if [GlobalControls]►AllowParallelAIQueues= (boolean) isn't added in rulesmd.ini AI does clones when it has multiple factories.
In the documentation it says that by default Ares don't let AI to do clones until you edit this new tag.

 I tested it in a single player mission and now in a multiplayer test.

Revision history for this message
Graion Dilach (graiondilach) wrote :

This might be correct.

An old comment I did about last February in one of my rulesmds...

"AllowParallelAIQueues=yes ; I'm pretty sure yes is the default instead of what the manual says, but who knows? - Graion"

On the other hand, I think the manual should be rewritten and we can leave the default as-is.

Revision history for this message
DCoder DCoder (dcoder1337) wrote :

Yep, the source says that my original implementation of this defaults to yes... https://github.com/Ares-Developers/Ares/blame/v02/src/Ares.GlobalControls.cpp

Someone else should feel free to decide the appropriate resolution for it.

Changed in ares:
status: New → Confirmed
Revision history for this message
AlexB (alexander-b) wrote :

Usually I would use a default value that corresponds to how original YR does it, so modders have to opt in. If it is a bug apparently, defaults should be fixed. If people relied on the original behavior, it shouldn't. All three apply. Safest way would be to change the manual instead of the default value.

Revision history for this message
Renegade (renegade) wrote :

Since yes should be what stock YR does, I vote for updating the manual rather than the code - that way, if the modder does nothing, Ares behaves as usual/expected.

Changed in ares:
importance: Undecided → Low
milestone: none → 0.2-rc1
Renegade (renegade)
Changed in ares:
assignee: nobody → Renegade (renegade)
Revision history for this message
Graion Dilach (graiondilach) wrote :
Changed in ares:
status: Confirmed → Fix Committed
Revision history for this message
FS-21 (jagarni1983) wrote :

(to confirm that somebody revised that manual update)
I'm using "AllowParallelAIQueues=no" to disable that factory cloning "feature" just like is specified in the updated manual.

Revision history for this message
Renegade (renegade) wrote :

Thank you! If you have the power to, you can also set the status to released yourself. :)

Changed in ares:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.