Opened 3 years ago

Closed 2 years ago

#6213 closed defect (fixed)

Autoqueue issues

Reported by: wraitii Owned by:
Priority: Should Have Milestone: Alpha 26
Component: UI & Simulation Keywords:
Cc: Patch: Phab:D4144

Description (last modified by wraitii)

  • When the production queue is modified, the auto queue should untoggle

- Autoqueue ignore entity limits and allows training infinite heroes.


Depending on difficulty, a way out is deactivating the feature for now.

Change History (11)

comment:1 by wraitii, 3 years ago

In 25753:

Prevent ProductionQueue.AddItem for going over the entity limits / Fix autoqueue being able to train infinite heroes.

Autoqueue can ignore entity limits like Heroes, because AddItem does not check for them, only commands.js. This changes that, fixing the autoqueue.

Reported by: Player of 0AD

Tested by: langbart
Refs #6213

Differential Revision: https://code.wildfiregames.com/D4133

comment:2 by wraitii, 3 years ago

Description: modified (diff)
Priority: Release BlockerShould Have

RB bug fixed.

in reply to:  description comment:3 by Freagarach, 3 years ago

Replying to wraitii:

  • When the production queue is modified, the auto queue should untoggle

That is a matter of taste. I regularly modify the queue and it would be not nice to have to enable it again thereafter, IMHO. Ideally we'd have more elaborate queue-modifying actions, like click and drag the order of items.

comment:4 by Stan, 3 years ago

Patch: Phab:D4144

comment:5 by wraitii, 3 years ago

In 25779:

Improvement to autoqueue usability

The graphical interface will show autoqueued units as 'ghost' units so the player gets immediate feedback on what's happening.
The autoqueue disengages if it runs out of resources or entity limits: this reflects that the autoqueue cannot resume on its own.

By changing the spot where the autoqueue pushes items, the autoqueue no longer 'buffers' one unit in advance, but remains very slightly less efficient than manual queuing. This also prevents cheats from leading to a ton of units accidentally being created.

Accepted By: Freagarach

Refs #6213

Differential Revision: https://code.wildfiregames.com/D4144

comment:6 by wraitii, 3 years ago

Milestone: Alpha 25Alpha 26

comment:7 by Freagarach, 3 years ago

Autoqueue should be disabled when changing owner. PetraAI doesn't disable it when captured and it messes up her planning.

comment:8 by Freagarach, 3 years ago

In 25819:

Disable autoqueue when changing ownership.

PetraAI can't cope with that and it may be a suprise for other players as well, when a captures production entity has autoqueue enabled.

Differential revision: https://code.wildfiregames.com/D4185
Refs: #6213

comment:9 by Silier, 3 years ago

Component: Core engineUI & Simulation

Is it fixed?

comment:10 by Stan, 3 years ago

As far as I know the first part of the ticket isn't.

comment:11 by Freagarach, 2 years ago

Resolution: fixed
Status: newclosed

The second part is fixed, the first part is won't fix, IMHO. As for improvements, they are certainly possible. See e.g. #6278.

Note: See TracTickets for help on using tickets.