AFTERMATH STATISTICS WITH GAME INFO/ACHIEVEMENTS

ubernaut

LaFayette

I wonder if it is still behind a feature flag, try going to settings and turn on beta features.

ubernaut

@LaFayette that was it i was wondering why i never could find what people were talking about

ebbe

@ubernaut : okej, where can I find these settings? Surched but without result in latest version... so still don't know what people are talking about

ubernaut

@ebbe check your engine prefrences

ebbe

@ubernaut said in AFTERMATH STATISTICS WITH GAME INFO/ACHIEVEMENTS:

check your engine prefrences

THNXS!

ebbe

@ebbe Looks like a good start, unfortunately I don't have the skills to
contribute to this features development, but I hope one day it can be possible to implement as popup in games end

djabwana

This is also closely related to this post about improving the message when victory conditions are met.

Thanks for the tip about enabling the beta features to see the stats. Here's what the current iteration looks like (I agree a great start!)

LaFayette

I think we may have neglected to remove the feature flag for the statistics, looks like that could be done to make this generally available.

djabwana

I would like to compute the backing data off the UI thread b/c it does freeze the game when you load it. I would also like to use the faction's colors from the map .properties instead of assigning random colors. I'll take a peek and see if I can do it

djabwana

Alright I've made some progress and will open a PR soon. I also want to show % change in addition to absolute value and will try to add that. This is now using the faction's colors from the map, which is "auto assigning" a color for the Alliances, which isn't ideal... I was thinking about splitting aggregate alliance data up from individual factions into separate charts (or a toggle) anyway.

Schulz

I liked the idea. But still not sure which colours would represent best the alliances? Maybe mix of all alliance member's colours? Once time i had tried it which resulted very unique colours.

LaFayette

There is a hash function for assigning player colors that are otherwise unspecified. The same function could potentially be used for alliances which would at least give a consistent color.

Cernel

I think this is a neat feature to develop. Maybe it should pop-up automatically after you accept the victory notification.

@djabwana said in AFTERMATH STATISTICS WITH GAME INFO/ACHIEVEMENTS:

"auto assigning" a color for the Alliances, which isn't ideal...

No idea if possible, but I would have it as a dashed line without spaces between the dashes, where each dash corresponds to the faction turn within the round shown by the X axis and is coloured as that faction (so, the alliance line would be a dashed sequence of all colours of the alliance factions).

By the way, the X axis should start as 0 and have value 1 at the end of the first round (not at the beginning), if it is continuous, or having the number 1 in the middle of the space representing the round, if it is discrete (Roman numerals would also strongly hint discreteness).

I also suggest fixing the bottom-left corner of the graphic always on the 0 (so you never have negative values and you don't cut any positive space, no matter if unused (as, eventually, some faction is likely to get killed, going to 0, anyway)).

Finally, you should apply the "Impassable" colour as background (on the assumption that the map-maker got it distinctive enough) or somehow determine a good colour based on the colours: you cannot assume maps not having the white (or next-to white) colour for at least one faction, as indeed some of them do. I realize that not every map has the impassable colour.

@djabwana said in AFTERMATH STATISTICS WITH GAME INFO/ACHIEVEMENTS:

I was thinking about splitting aggregate alliance data up from individual factions into separate charts (or a toggle) anyway.

I think best would be a toggle for having, on the same chart, only the factions, only the alliances or both at the same time (so not really a toggle, as there would be 3 options). In theory, I could make a game with 100 factions per alliance, where each faction line would be pushed so much on the bottom to become almost indistinguishable, so I think the default should be either factions-only or alliances-only.
Of course, you can have alliances having only one faction. In this case, they should figure in both the factions-only and the alliance-only displays, but they need to be shown only once, as faction/alliance, when showing both. This matter is also relevant for when you happen to have only one faction remaining on the map for the alliance (you will have to decide whether the alliance or the faction overlaps, so hides, the other).

djabwana

Thanks everyone for the feedback!

I switched out the "fixed" color palette to one from https://colorbrewer2.org/
I changed the random assignment to be a fixed brightness
I realized map designers can assign an alliance color in map.properties so I don't think we need to get fancy- map makers can pick a color for the alliance if they don't like the auto assigned one (which should be more distinguishable and aesthetic now)
Data loads off the UI so you can interact with the UI while it's loading
You can toggle players, alliances, or both and both is split between 2 axes
The window itself is no longer a JOptionPane but a normal JFrame that can be resized
I took it out of beta in the menu

My progress is here:
https://github.com/jdimeo/triplea/tree/game-stats-rework
Still have to clean up checkstyle stuff before I can open the PR

Loading off UI thread:

Players TUV:

Alliances Production: Revolutionaries came from map.properties whereas the other two were auto-assigned.

PUs for both: Kinda a mess, but if you don't have two y axes the lines of the players will always get smashed by the alliances

djabwana

@Cernel I'm not sure if x = 1 means "start of game" or means "after the first round". I'm focusing on the charting aspect and not the data collection. Maybe someone else can weigh in on that. I'm pretty sure it's what we want though b/c I see that USA has 0 PUs which is true at the start of the game, so I think 1 does really mean 0 in the sense that you were thinking. I did force Y to zero though for all charts.

djabwana

More generally though, I would love to pop this up at the end of each round and also work it into a more "newspaper" format with notable battles, or objectives achieved, or VCs captured, etc. that round as a recap and a way to celebrate achievements. So I also propose we rename this to "Round Summary" or something to start going in that general direction instead of a more "Game Statistics" direction (though I love the current charting and it should definitely be a part of this long term!)

Cernel

@djabwana Yes: I'm pretty sure the "1" on the Axis means the start of the first round. To make it clearer, I suggest either moving the numbers on the centre of the periods (so, for example, the "1" would be in the middle of where now there are the "1" and the "2") or move to an actual continuous enumeration, in which case the current "1" should be "0", the current "2" should be "1" and so on.

Either can be used. For example, I believe we use continuous for persons years (so I have 1 year 1 year after I was born, not in the moment I was born) and discrete for history years (so it has been 2,000 years when the year turns 2001, not when the year turns 2000, so one which was born in the year 2000 is not a so-called "millenial", for example, because the year 2000 is the last year of the past millenium).

LaFayette

@djabwana FWIW a joy of Swing is in theory it's bad practice to use multiple JFrames. In particular and in practice we saw problems in TripleA where on some systems and some JDK versions new Jframes appeared behind the game JFrame. That behavior was not consistent, but on those systems it typically was and certainly generated a number of bug reports.

Cernel

@LaFayette Off-topic, but the problem has never gone away. Currently, if you open the Notes or Unit Help window, it will be instantly hidden behind the map as soon as the focus get to it (typically, for someone else starting a battle). Normal users will likely just think that it has disappeared so will proceed opening a second one and so on.

Reference:
https://github.com/triplea-game/triplea/issues/8607