Project

General

Profile

Actions

Feature #1505

open

MaxTopUnitsOnTile

Added by Marko Lindqvist 5 days ago. Updated 1 day ago.

Status:
In Review
Priority:
Normal
Category:
General
Target version:
Start date:
06/10/2025
Due date:
% Done:

0%

Estimated time:

Description

Add requirement type "MaxTopUnitsOnTile". The difference to old "MaxUnitsOnTile" is that transported units are not counted, only topmost transporters.


Files


Related issues 1 (1 open0 closed)

Blocks Feature #1517: granularity: Allow max 4 units on tileIn ReviewMarko Lindqvist06/12/2025

Actions
Actions #1

Updated by Marko Lindqvist 5 days ago

Actions #2

Updated by Alexandr Ignatiev 5 days ago

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Actions #3

Updated by Marko Lindqvist 5 days ago

Alexandr Ignatiev wrote in #note-2:

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Yes. It's more correct for MAXTILETOPUNITS than MAXTILETOTALUNITS (renamed MAXTILEUNITS), for which the doc has been changed.

Actions #4

Updated by Alexandr Ignatiev 5 days ago

to

Marko Lindqvist wrote in #note-3:

Alexandr Ignatiev wrote in #note-2:

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Yes. It's more correct for MAXTILETOPUNITS than MAXTILETOTALUNITS (renamed MAXTILEUNITS), for which the doc has been changed.

Hmm... If I do not miss some patch, it said about number of units present on the tile(s). Really, the difference between "number of units" and "total number of units" does not sound very clear. At least we are used to that a unit that is inside Y which is in X is still present in X.

Actions #5

Updated by Marko Lindqvist 5 days ago

  • Status changed from In Review to In Progress
Actions #6

Updated by Marko Lindqvist 5 days ago

I changed MaxTopUnitOnTile reqtext.c messages to talk about units directly on the tile. Is this better?

Actions #7

Updated by Alexandr Ignatiev 4 days ago

Marko Lindqvist wrote in #note-6:

I changed MaxTopUnitOnTile reqtext.c messages to talk about units directly on the tile. Is this better?

Well, I'd use something like "untransported" or "(except ones in transports)", but tastes differ, it's already something :)

Actions #8

Updated by Marko Lindqvist 4 days ago

  • Blocks Feature #1517: granularity: Allow max 4 units on tile added
Actions #9

Updated by Marko Lindqvist 2 days ago

  • Status changed from In Review to In Progress

You're right "untransported" would be better. "directly on tile" is somewhat ambiguous when it comes to units inside cities or extras (fortresses).

Actions #10

Updated by Marko Lindqvist 1 day ago

Marko Lindqvist wrote in #note-9:

You're right "untransported" would be better. "directly on tile" is somewhat ambiguous when it comes to units inside cities or extras (fortresses).

Changed the patch accordingly.

Actions

Also available in: Atom PDF