-
Notifications
You must be signed in to change notification settings - Fork 393
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.6 Refresh maps-list/maps/ only 173 listed out of currently 315 repositories #12647
Comments
@TheDog-GH It's normal that the repositories are more (they include currently non-supported map-skins too), though I'm surprised that there are actually as many as 142 of them not being available maps. |
@Cernelius Ah yes, they will not match, thanks! With my new powers of access to some of TripleA, I guess that this update was ran 2 years ago and mainly Roger has been adding maps over the years, but Im new to this and my deductions might be wrong. |
Yeah @RogerCooper's been adding tons. I wonder whether he intends some of these to remain in the repository but outside download list or he plans eventually to add them all to it. For example, "alexander350". |
Last time I chatted to Roger about this he was not going to bother with 2.5s However he did say that he was making them 2.6 Download ready by including a map.yml file like alexander350 |
I have only a few maps left to upload to Github. Then I will starting reviewing the repositories and them triplea_maps.yaml when appropriate. (I will try to screen out superseded, duplicate and unplayable mods). |
Barring any kinks, there are some updates that should resolve this issue: (1) download-all-maps script is invoked daily. Added a cronjob I'm not sure it'll run 100% smooth, TBD. (2) bots restart automatically when a game ends. Instead of going back to staging screen, the bot halts & the system restarts it automatically. Bots, on-startup, read new maps. Without this, there was the other problem of how to get bots to read the new maps after they have been updated. In sum - map updates should flow every day and will be picked up on bot restart (which is now whenever any game ends) |
Having left it a couple of days, this is still not working? Still finds 173 maps |
@TheDog-GH I don't understand what is requested here. Those are the maps in download list, so their number should match the number of maps in that list, not necessarily the number of maps in the repository (beside the fact that the repository contains map-skins too, so it would not match even if all maps are listed). |
Also, I'm not sure why we are talking about bots. This feature was not about bots until @DanVanAtta introduced them. Am I missing something? |
@Cernelius good point, I confused this issue with #12517 The process for updating the maps on the website is certainly in trouble since we will be changing how maps are indexed. The update process from triplea_maps.yaml to website was explicitly shut down back in 2022 in anticipation of having the system be reworked. For reference, this SHA: 9350ca2 The open questions are whether to actually restore this update process (and if we don't, that implies we'd remove the 'maps' page). Second question, is how & when to do the updates. As-is, I don't know if any efforts are worthwhile here compared to focusing on solving the longer term issue. |
My 2p is to replace it with a static page, with 5-10 chosen maps, maybe from here With suitable text to show off TripleA and that its maps can do history, fantasy & Sci-Fi. |
Not a bad thought. Some other possibilities: (1) Github does provide a web API for listing repositories, we could bring back the map-splitter script with that as the data source instead 'triplea_maps.yaml' .(2) similar to 1, except use the maps server which will have a similar API that provides a list of maps - perhaps easier to parse. |
There are Github repositories which do not contain any scenarios. These seem to be from abandoned projects. There seems to be some confusion in terminology. I use the following terms: Mod - A folder on Github or your local computer that contains all the files needed to play one (or more scenarios) Scenario - A specific entry showing up on the menu that defines a specific playable game Map - The graphic and text files that define the game's on-screen map. In older versions of TripleA, the map could be defined specifically to use another mod, now you need to copy the map files from the mod. I am almost finished with posting my mod collection to Github. I was planning on adding the mods to triplea_maps.yaml, as that is the currently functional area, |
@RogerCooper I'm almost certain that what you call "mod" is a "map" and what you call "map" may be a "skin" (except that it comprises everything outside the "games" folder so for example sound files too). However, I'm not sure what you mean by "text files": if you comprise the "XML" ones, then what you call "map" is indeed a "map", but then I'm not seeing much of a difference with what you call "mod" because, if you take out the "map" file from a map, that already "contains all the files needed to play" its games. Having a "map" file as a sub-folder is something not necessary, and in the past there was nothing like this, actually: what is now the "map" file was named with the map name and was the first level folder of the map itself (and this construct is still supported by the program). No problem with "scenario" if you want to suggest that name, but of course that has been called "game". |
I would say that TripleA is a game. The use of 'map' is confusing as in common usage a map is representation of geography. TripleA uses maps, but a mod has lot more in it than just a map. I am coming at this as long time board wargamer, where games have scenarios and sometimes multiple maps. |
If we want to propose clarifications or changes of terminology and their adoption and documentation as official (documenting them somewhere in an official TripleA dictionary) so to better define the TripleA jargon in order to facilitate communication, I suppose that should be attempted in a dedicated discussion. About
are you sure that you are not looking at map-skins? |
By the way, I still cannot understand what this issue is asking to be done. Like, what exactly is needed for this issue to be closed as completed? I'm at a loss. |
Re: glossary FWIW: https://github.com/triplea-game/triplea/blob/master/docs/development/glossary.md But, it's worth keeping in mind that TripleA represents multiple bounded contexts. In other words, the word "map" really depends on who you are talking to & when. A player vs a map-maker vs a developer can all have three equally valid yet different answers. And moving right along... The list of maps at: https://triplea-game.org/maps-list/maps/ , has not been updated for some time. At first glance, we could resurrect the "YAML splitter", but the file "triplea_maps.yaml" is due to be retired "very soon." That does not represent a long term solution. So, we need to do something like add a once-a-day github action job that will do something similar as the YAML splitter, but instead needs to fetch its data from a web service call to either Github's webservice API, or one we provide via the "maps server". The latter is likely going to be easiest to avoid the issues of parsing data in a more elaborate manner. Specifically our own custom API can provide an endpoint that gives exactly the needed JSON, which in turn is very little parsing for the YAML splitter to do. |
https://triplea-game.org/maps-list/maps/ says 173 maps (Find view -1=173)
https://github.com/orgs/triplea-maps says 315 repositories
maps-list/maps needs to be refreshed to match.
Dan I think this one is yours?
The text was updated successfully, but these errors were encountered: