In D3213#142287, @Stan wrote:

Nice! Will try to test it soon (not tonight as something came up) i think you forgot to remove the entity.pm file :)

Fixed invoking with -t in windows (also is python, invoke a module)

Included logging, and the bad separator fix

In D3213#142105, @Stan wrote:

Also if I may, is it possible to have the prints as the script goes, instead of all when it fails? Maybe this is a windows bug?

Explicit utf8 opening.

Changed path handling, added mul_round

"fixed" path for windows

I tried and it works. However it generates 3 warnings, this patch below fixes them.

Tested with nomad on latest svn. It works.

Fixed Tooltips not working, store just generic, refactored cmpResourceGatherer.AddToPlayerCounter

Fixed preinc and predec.

Fixed tooltip text.

Addressed latest comments, refreshed screenshot.

Included tooltip (mouseover) support, thanks to @Nescio. Hoping is correct to handle translation like this.

In D3155#140914, @Nescio wrote:

So I've tried your patch. Two things:

• Only the first digit is in the bottom-right corner of the resource icon, which means if the number is ten or more, the first digit is on the icon, the other right next to it; thus the 4 in 4 is aligned with the 1 in 13 (it ought to be with the 3).

Added more space since we moved right 4 pixels for readability.

Renamed to AddToPlayerCounter, RemoveFromPlayerCounter. Addressed latest comments.

Addressed latest comments.

I tried this patch and I confirm it fixes the "nomad" option in my tests.

Addressed latest comments. Cleaned some cruft.

Removed handling of traders and trickles, just gatherers. Starts counting earlier as gatherers. Addressed some comments, cleaned and refactored.

Cleaned a bit, some renaming, some comments addressed.

Seems to work, death supported, some comments addressed.

In D3155#139549, @Angen wrote:

As resource trickle does not count as gatherer, it is out of ticket.

In D3155#139553, @wraitii wrote:

In D3155#139549, @Angen wrote:

Now units returning resources and going back do not count while walking.

Mh, true, I didn't consider that. I think we'll want to count those, but it will impose adding new calls in RETURNRESOURCE.

In D3155#139456, @Stan wrote:

It doesn't take ResourceTrickle into account though, not sure that's intended.

Simplyfied, removed sets of gatherers, tied the counters to resourceSupply. Still not work on death of the unit, and really fast switching since resourcesupply handling, left debug code in to try to work this out.

In D3155#139116, @wraitii wrote:

There is a meta question which is "should we show gatherers or the sum of gather rates?" I'd probably prefer the latter.

Still missing moving the code from Player, but fixes some comments, it shows total gatherers under population icon too, which could be useful. Still not clear how to handle properly deaths, this patch should also handle replies better, although not ideal.

In D3155#139047, @wraitii wrote:

(still suggesting moving all those to resourceGatherer)

The UI works nicely now, the replay part needs some advice, not yet moved off "Player".