TripleA development
-
@LaFayette thanks for closing my PR as you mentioned. In that case, I was purposefully trying to sidestep all the things that we've been discussing here by making it totally "under the radar" - no javadocs, no comments, nothing- just actually making the code the way it should've been from the beginning (IMO) since that data structure is both a list (reading the list of points from the file) and a "lookup" structure (to find the point associated with the name). In other words, the burden falls on the original developer to comment why
LinkedHashMapwasn't used and original order was lost, since that would be the default assumption/expectation.With all love and respect to @Cernel (he always has good points/ideas/feedback/context, but despite that...) it was really frustrating to have my intentionally minor/secret/quick/minimally-invasive PR get mired down in long discussion. So I see both sides of these issues- ease of contributing vs. burden on maintainers. I think it's just that TripleA has become an enterprise-level codebase (there are some great and powerful features in there) but maintained by hobbyists and we all have "day jobs"...
For anyone who has invested time in it, thank you! it's been a great venue with keeping up with old friends I used to play A&A with in high school.
-
@lafayette said in TripleA development:
Though, the prerelease server had been suffering from disk space issues; this caused disk to fill up and the database then to crash and be unavailable for any requests to it thereafter (even if disk space were then freed). To fix this, the DB needed to be restarted and this was being done for some time.. I recently found, about a week ago, what looks like old map files that were taking up around a third of the disk. We have the bots download all maps, all maps are about 5GB or so, and there was an extra copy of this floating around. Without that extra copy we should have breathing room and the problem gone.
Oof. Double whammy. If you need a hand, I professionally live and breath this type of architecture and services. I updated my profile with my LinkedIn if you need referrences.
The version change is a red herring; it should have been fixed for the config panther was using for previous 2.6 versions as well. It just so happened it was fixed after downloading a new version. If there really is a difference between versions downloading from prerelease explicitly, that would be very interesting.
No difference in the 2.6 versions. I confirmed that this afternoon.
Side-bar, is there anyway we can move this 2.6 launch related conversation to the 2.6 conversation thread? https://forums.triplea-game.org/topic/2999/2-6-release-getting-close-need-volunteers-to-help-beta-test-2-6
Absolutely.
-
@magicstyck hahaha if we all joined up and started a software company together, it would be amazing
-
@djabwana said in TripleA development:
@magicstyck hahaha if we all joined up and started a software company together, it would be amazing
:beaming_face_with_smiling_eyes:
That made me laugh out loud :face_with_tears_of_joy:
-
What about issues where there's nothing more the person who opened it can do; and the only people who possibly can address the issue are very busy (eg you)? I mean, I tried to push the long-standing problem affecting the Global map unit transfer in lobby bots; but you're very busy, and poking you repeatedly doesn't change that you're busy, and it's not something that can be fixed by someone else since it appears to be about deployments.
I just don't see any good ways to handle it; sometimes priorities have to be made and certain issues will just get put on indefinite hiatus because the only people who can address them are busy.
-
I feel like we're all mostly at least on the same page. Some problems certainly remain, but our perspectives are more aligned now - that is valuable.
I think @MagicStyck & @frigoref should gain system access to at least the pre-prod environments if not also prod so that they can help there.
I do strongly prefer having squash merges for a better history, and certainly at least rebase merges so that the history is linear. We have had several bugs where we were only able to track the problem down after bisecting the history. If the history were non-linear, we would no longer be able to do that. Some problems are amazingly subtle, I'm sure we would have had to roll back a years worth of commit if the history were non-linear. Perhaps the answer is to more readily give write access to those who want to contribute more than a single PR. This way those that are doing the bulk of a PR can be the one doing the merge and get attribution. As a complement and/or alternative - perhaps in the release notes we could explicitly mention the contributors since they may not appear in the git history properly.
-
@zlefin There is a 'high impact' label or something like that to note issues that are more significant. Adding that label where appropriate would presumably help. I've been avoiding a bit having a lot of priority labels lest we fall into the trap of adding a lot of "work about work" that is not actual 'work'. Suddenly labeling and assigning priorities becomes a job, which is not very helpful if all we do is bicker about priority and shift priorities around without actually accomplishing those priorities. We also run into the problems of "important to me, but not you, so where is the priority?"
If a map is majorly broken, eg: the unit transfer functionality which I think has broken on just about every release over the last 5 years, is a major feature of some maps. It would rank pretty high to fix that.
-
Well, with global it's only moderately broken; the problem only applies in lobby bots, and most people are satisfied with fixing it via using edit every round. It's infrequent enough that doing so isn't too onerous. It does make TWW very hard to play in lobby bots, as there's alot of unit transfer triggers in it. Should I apply some label to the pertinent bug report on the matter? Since it's something that afaik only you can address, you should be the one deciding on the priority level.
-
So far as taking care of problems goes, there are a few map issues/PRs here that could be looked at, and several can probably be merged. They may not be in the main game, but they do make issues in the main game when they have problems.
-
@ff03k64 I don't understand "They may not be in the main game, but they do make issues in the main game when they have problems". How can they "make issues in the main game", when they are not in the main game?
Should a rule guru look over them first before a developer starts? -
@LaFayette Can you create a guide on how to test such bot issues? I would have no idea how to analyze them.
-
@beelee, @Panther , @RogerCooper
Would you please provide your comment on the to-be TripleA issue process flow I have tried to draw?
-
@frigoref this is awesome! but I think you mean "Rules clear?" "No" goes to Guru
-
What @djabwana says.
Also I stumble a bit over the "rules clear?" question.
In more than a decade of rules discussions I have learnt that despite rules might be "clear" to someone that "clearness" must not necessarily cover reality.
So maybe it would be a good idea that someone familiar with the rules looks at every issue that touches rules questions (what actually has been practiced for years, IIRC).
-
@frigoref Good basis to clarify who should do what!
If you want to store graphs in a markdown (.md) file, you might want to use mermaid or PlantUML, see https://www.jetbrains.com/help/idea/markdown.html#preview
-
@rainova I have played around with it a bit, but it I think the repeat loops are always on the wrong side causing the diagram flow to be more confusing that it actually is:
-
@frigoref You should add an intermediate step after the "rules clear?" part.
I think the current label is almost worthless as it is usually added without clarifying what needs clarification, so it is basically an encouragement to add more and more text to the issue trying to catch whatever is being unclear to whoever.
If a rule is not clear to everyone, the person that feels so should clarify exactly what needs to be clarified and then you can actually clarify the rule, so you need an intermediate thing between the "rules clear?" and "clarify rule" steps.
Moreover, the "clarify rule" step should direct back before the "rules clear?" step, because having clarified a rule doesn't imply that is actually fully clarified. So the clarify rule should direct to "update issue".
Moreover, I've no idea how an issue can be complete if it is not clear, but maybe you mean that just satisfies the minimal requirements of reporting. I think you should clarify what you mean by "complete".
-
@cernel You are right this process flow is not detailed enough for what exactly to do in which situation. However, that was not the goal as this would limit the main goal of clarity of different parts and participants for the whole process.
Basically the Issue Dispatcher should check and provide labels based on which the other roles should take actions. For sure just a label is not enough and additional request/questions should be also added by him/her.
-
May I ask for your help to create a profile for each of the following roles we have identified so far:
- Developer with merge rights ( @RoiEX, @LaFayette )
- Developer w/o merge rights ( @RaiNova, me )
- Issue manager ( @beelee )
- Map makers ( @RogerCooper )
- Rule gurus ( @Panther )
- Forum moderators ( @Cernel )
If someone would like to chip in, please feel free to come on board and assist.
My idea of a profile (that we could gather in a new profile folder in https://github.com/triplea-game/triplea/tree/master/docs would look as follows:
Title: Developer w/o merge rights
Description: Foot soldier who works on the code to fix bugs and add new features.
Skillset- Experience with object oriented programming language (preferable Java)
- github account to contribute
Contributions
- Bug fixing
- Implementing new features
- Code cleaning
- Code review
How to become one: Setup your system and check our development processes
Before we start: What do you all think about this structure?
-
@frigoref you could have Panther and Cernel for rules. Cernel would qualify for maps as well and probably Panther too. My guess is new devs don't have merge rights is because it could mess stuff up if they went solo mode. My impression is you guys are all cool and probably wouldn't be a problem, but as i've said before I just update the maps and don't mess with anything else.
Idk, It's good to have a "structure" . It seems there already is one of sorts though ? Idk
Foot soldier seems a bit harsh : ) Reminds me of my buddy deploying when I asked him his job. "Bullet Stopper" was his response. Fortunately he made it back ok. Or not blown up anyway.
Edit
But yea it'd be fine by me. Didn't mean to sound overly negative or anything : )
Hello! It looks like you're interested in this conversation, but you don't have an account yet.
Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.
With your input, this post could be even better 💗
Register Login