This Trac instance is not used for development anymore!

We migrated our development workflow to git and Gitea.
To test the future redirection, replace trac by ariadne in the page URL.

#6750 closed defect (fixed)

After queued construction of several fields, woman becomes idle

Reported by: Langbart Owned by: Freagarach
Priority: Release Blocker Milestone: Alpha 27
Component: Simulation Keywords:
Cc: Patch: Phab:D4960

Description

to reproduce

  • start a game with the Han select at least 4 women and task them to build two rice fields
    • You can also choose another civ, but then you would need 6 women
  • hold the shift key pressed while giving the order
  • when the second field is fully constructed, one woman becomes idle

bisect

[27559]

Attachments (1)

idle_farmer.gif (665.7 KB ) - added by Langbart 19 months ago.

Download all attachments as: .zip

Change History (9)

by Langbart, 19 months ago

Attachment: idle_farmer.gif added

comment:1 by Stan, 19 months ago

I think the bisect is incorrect. That commit even states it's gonna do that. It's an improvement over before where units would not build farms at all

comment:2 by Langbart, 19 months ago

[27558]
Behaviour: Build foundations in the queue, and start farming on all fields.

[27559]
Behaviour: Build foundations in the queue, and all units that fit on the last field start farming, the rest become idle.

It's an improvement ...

in reply to:  1 comment:3 by phosit, 19 months ago

In your example the last woman will:

Bevore [27559]: Constructing the fields -> try to garther from the most resent field (fail) -> try to work on the other field.

After [27559]: Constructing the fields -> peek if it's possible to work on the most resent field (it's not possible) -> search for other fundations to construct (fail) -> become idle.

comment:4 by Langbart, 19 months ago

[27558]
If I have 8 Han women on the berries and they are almost done, I could select all the women, place 3 fields while holding down the shift key, and not have to worry about any of them ever becoming idle.
[27559]
Select all 8 women from the field, try to select 3 of them and give the queue order to build a farm when done. Try to select another 3 women do the same and another 2.


  • more clicks, higher probability of idleness, and harder to micro manage your women
Version 1, edited 19 months ago by Langbart (previous) (next) (diff)

comment:5 by Freagarach, 19 months ago

Yeah, the moment I saw the title of the ticket I saw the codepath in my mind. ^^'

Last edited 19 months ago by Freagarach (previous) (diff)

comment:6 by Freagarach, 19 months ago

Owner: set to Freagarach

comment:7 by Freagarach, 19 months ago

Patch: Phab:D4960

comment:8 by Freagarach, 19 months ago

Resolution: fixed
Status: newclosed

In 27567:

Revert r27559 / rP27559.

Introduced #6750, reported by @Langbart, and quite hard to do right.
Fixes #6750

Note: See TracTickets for help on using tickets.