Student Group Studying TripleA

ebbe

@thedog @jtkenny
1. Displaying how many of a unit can purchased and blocking the purchase of too many units see the image below, the units with a ≤
2. calculating how many units can be placed this turn, in this example 11.

These are on the top of my Wishlist too !!!! ; )

jtkenny

@beelee This is the issue we selected to try to work on, although so far we have been unable to replicate the error: https://github.com/triplea-game/triplea/issues/11097

So far we are thinking that maybe this is because we are on the 2.6 version of the game, and this may be on a later versions, but we also haven't been able to figure out how to move forward to this version.

jtkenny

@lafayette Yes our entire group have completed these steps for the most part. I have been able to run the application in IntelliJ on my computer, and I think everyone else has as well.

TheDog

@jtkenny said in Student Group Studying TripleA:

but we also haven't been able to figure out how to move forward to this version.

Not sure what you mean, but I think you need to install multiple versions of the TripleA game
So download a Triple version, but when installing it, use the version as the install path
eg install to TripleA-2.6+14163
like;

and therefore download the source of the version that you want to change.

Im not sure the one issue you have selected, is actually an issue. I'm sad and have no life and I use TripleA daily and I have not seen this error message, speaking as a xml coder of many maps and many errors.

Being selfish the biggest impact you could make for us TripleA-ers is in the 2nd post of this thread, as lots of maps have this style of unit restriction.

If you met this restriction and gone over a unit limit, TripleA resets all your purchases and you have to start over, not a good experience for a player to have.

LaFayette

@jtkenny There is a tag pushed on each release that matches the release version number. You should be able to checkout the exact version via: git fetch upstream; git checkout 2.6+14163 (I'm assuming you named the 'upstream' remote as 'upstream', git remote -v should indicate the name to use).

The version in the error report does correspond to the game-client version that sent the error report, seemingly this (issue #11097) should be something that you can reproduce in 2.6.

It might be most fruitful to see how and where returnFire could be null. That might then give you a path for reproducing the issue.

Re: local dev setup

Just want to be sure that your team is all able to run verify script and not just launch the game from IDE. The verify script is useful since it runs all tests & static checks in one-go. It has more requirements then just running the game since you need to install docker (which then provides you with a local databse). If that works, then you'll be able to launch all components of the stack and not just the client application. To add to that, verify is particularly useful for PR verification - it can be quite endless to push updates and wait for the build to fail on relatively random issues compared to running verify locally (which is all in the name of reducing developer iteration time)

LaFayette

@jtkenny for issues to work on, also please consider the problem tracker: https://github.com/orgs/triplea-game/projects/3

The 'fix sooner' column has some relatively tractable items that are all relatively high impact. The 'standard fix' items are all similarly good items to pick up.

Even if the 'problem tracker' is not as complete as it could be, I did want to point this out as another reservoir of items that could be picked up.

LaFayette

One more item, it's a bit easier than ever to find the '2.6' error reports, we added some labels: https://github.com/triplea-game/triplea/issues?q=is%3Aopen+is%3Aissue+label%3A"Error+Report"+label%3A2.6

jtkenny

@lafayette I've been trying to run the verify script, but the the database startup times out and aborts and so far I have been able to run it.

LaFayette

@jtkenny I would guess the DB is not starting.

The startup script is mostly a wrapper around this command:

  docker run \
    --rm -d \
    --name="database" \
    -e "POSTGRES_PASSWORD=postgres" \
    -p "5432:5432" \
    -v "$(pwd)/$(dirname "$0")/sql/init/:/docker-entrypoint-initdb.d/" \
    "postgres:10"

A quick debug would be to run without the folder mount at first:

  docker run \
    --rm -d \
    --name="database" \
    -e "POSTGRES_PASSWORD=postgres" \
    -p "5432:5432" \
    "postgres:10"

Let us know how that goes

Lydia McCleary

Hi all, I am also a member of the student group. I have a few questions regarding TripleA's availability. Has TripleA ever gone down? How long is TripleA's typical down time (either when changes are being committed or unintentional outages)?

beelee

Hi @lydia-mccleary

The lobby goes down sometimes. Rarely for more than a few hours. Dan probably give you a more detailed answer.

LaFayette

@lydia-mccleary

Has TripleA ever gone down?

Yes. Though, in terms of availability, perhaps to some surprise it is quite high (out of the year, perhaps only a day or two of downtime). For better or worse, we do not measure the availability of the lobby, so a concrete answer is unavailable. Though, we are pretty quick to get reports about the lobby not being available, and there are not very many over the last 4 years. These are the ones I could find:

https://github.com/triplea-game/triplea/issues/10804 < deployment configuration problem, lobby database password got out of sync with lobby application

https://github.com/triplea-game/triplea/issues/9708 < regional issue relating to DNS resolution of our domain

https://forums.triplea-game.org/topic/1239/where-is-the-lobby-down-topic < linode "maintenance" (note, the person was asking 'where is the lobby down' topic in order to be able to post about the lobby being down!)

How long is TripleA's typical down time (either when changes are being committed or unintentional outages)?

I think the events above were all hours. I'm not aware of any occasional blips in the software, generally downtimes are discrete events (rather than say the lobby crashing on a recurring basis)

either when changes are being committed

Lobby needs a restart to pick up updates. Last time we did this was almost 3 years ago (maybe more even, lobby is running v2.4). We can have users log into a new lobby now without downtime but there is no shared state between different lobby versions. Meaning, a player might be the first to rejoin into a new version of the lobby and be the only one there.

For 2.6, we are pretty close to doing a lobby cut-over. It should be zero downtime. Some of this should play out in the near future. Happy to answer further questions.

jtkenny

@lafayette Sorry about the delayed response, we have been very busy with exams and writing an essay.

I ran the script you gave and it started the database, but I was still unable to run the verify script because it seemed to stop and then try to start the database again.

On another note, I left a comment on the issue we were going to start on. Is there a particular naming convention I should use for the branch we make for the PR?

LaFayette

@jtkenny More details on what is happening with the script would be helpful. Perhaps if you could add a 'set -x' to the top and then provide the output, it might be easier to debug what is happening. The verify script does start by dropping database data and then resetting it up for a 'clean' state. So that sounds consistent. It also sounds like the database is also able to start, which does leave me wondering where the verify script is terminating and/or going wrong.

It's pretty important for efficient development to be able to run PR build checks locally. This allows a person to do local debugging and more efficient iterations in tracking down problems. Not being able to run all of those build checks locally is a concern, as that should work. It implies we need to fix something so that it works and allows anyone to reproduce the PR build checks locally.

Is there a particular naming convention I should use for the branch we make for the PR?

Nope. The commit message is the important part, this link gives good outlines of how to write a good commit message: https://www.freecodecamp.org/news/how-to-write-better-git-commit-messages/

jtkenny

@lafayette Here is the terminal output when I try to run the script (with set -x):

The strange part is that Docker shows the server running, so I'm not sure why it aborts startup.

LaFayette

@jtkenny Thanks for the output. It's odd to see the 'git-rev-parse' stuff in there. The 'timed out' part is not good. What follows I can't really explain as I would expect a 'gradle' command to be the next thing to launch.

FWIW, I recently wiped my dev setup and am working from a clean install and have reproduced some of these problems.

When running the start docker command [1] on its own, I get a "image cannot be stopped warning". Seemingly the 'stopping database' step is not stopping the database properly.

If I comment out the 'start-docker-database' step in verify [2], then I do see tests kick off but many happen in parallel. This is not good since the tests that interact with database step on each others toes when deleting and setting up database data . So I then ran into various "table not found" errors when running tests

In the short term we'll probably need to disable the steps that run with a database. In the medium term there is an experiment with going 'multi-repo' (https://github.com/triplea-game/triplea/issues/11204) that would help this situation. Specifically if the lobby and its database wind up in their own repository, then our verification script becomes simpler and no longer needs to fire up a database. In turn, the lobby repo can have an explicit config to run tests in serial.

[1]

docker run \
   --rm -d \
   --name="database" \
   -e "POSTGRES_PASSWORD=postgres" \
   -p "5432:5432" \
   -v "$(pwd)/$(dirname "$0")/sql/init/:/docker-entrypoint-initdb.d/" \
   "postgres:10"

[2] verify:29 ./gradlew --parallel spotlessApply check -x $DATABASE_TESTS \

jtkenny

@lafayette Okay, just let me know what tests I should run or how to disable the database tests and I will make sure we get those run on our code.

jtkenny

@lafayette I am trying to make anew branch to put our changes in, but it seems like I don't have the permissions to push a new branch, do these need to be granted to me in some or is there a different way I should go about making a new branch?

TheDog

@jtkenny

Oh, oh, oh Im almost excited, your first PR is awaiting approval.

Zoom in & Out % is another quick win request;
2.6 can zoom In & Out, it would be good to show, when Zooming, that percentage on the right-ish of the status bar for say 3 seconds and then disappear, that is giving back the valuable status bar space.

Technology grid panel, another request?
To remove the Tech grid from the Players tab, so more nations can be displayed without scrolling to see the Alliance totals for Economic Victory or VC Victory.

The grid could then be accessed via a menu item, maybe ctrl+T ?
It would look a little like the ctrl+W Politics panel, flags and nation name for the X axis and Tech name for the Y axis. Maybe for researched, a cell would have a green background and a tick?

TheDog

@jtkenny
Here is another UI request?

It would really nice to have a centre button (to centre on the territory) against each one.
Currently you have to faff about to find the territory from the menu.

There are lots of other little UI tweaks, so pick the ones you would like to do.