Wikipedia:Bots/Requests for approval/MajavahBot 5
- The following discussion is an archived debate. Please do not modify it. To request review of this BRFA, please start a new section at Wikipedia:Bots/Noticeboard. The result of the discussion was
Approved.
Operator: Taavi (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)
Time filed: 11:14, Monday, July 31, 2023 (UTC)
Automatic, Supervised, or Manual: automatic
Programming language(s): Python
Source code available: gerrit
Function overview: Import ATP rankings from a PDF to a JSON page to be used by Module:ATP rankings and {{Top ten male singles tennis players by country}}
Links to relevant discussions (where appropriate): Wikipedia:Bot_requests#Updating_a_module_based_on_a_PDF
Edit period(s): weekly
Estimated number of pages affected: 1
Exclusion compliant (Yes/No): No
Already has a bot flag (Yes/No): Yes
Function details: The bot downloads the PDF file from the ATP website, parses the scores from it and formats it to a JSON file with details for the top 15 players in each country. The results for last week are also stored to display change information.
Discussion
[edit]I'm involved in this request and the template maintenance so I can't really reply here with my BAG hat on, but this will help to keep the 80+ navboxes that use this information up-to-date. of course, while holding my BAG hat, I will say that this is a straight-forward task which the bot itself can't really screw up and I can't see any issues ;-) Primefac (talk) 11:28, 31 July 2023 (UTC)
Approved for trial (1 edit). Please provide a link to the relevant contributions and/or diffs when the trial is complete. – SD0001 (talk) 12:08, 31 July 2023 (UTC)
Trial complete. See Special:Diff/1168042490. This update is a bit larger than future ones would be as the "old" data (used for generating the change numbers) was not fully up-to-date with the latest changes to the code. Taavi (talk!) 12:18, 31 July 2023 (UTC)
- I think the only question left is what to do about Russia and Bulgaria - players from these two countries have blanks in their country field and thus do not get ported over to the json. Is this enough of an issue to hold up the bot task, or is it something that can be worked out over time? Primefac (talk) 13:19, 1 August 2023 (UTC)
- I think that's fairly simple to add later. The simplest way I can see is that we create an on-wiki JSON page that the bot can read country "overrides" from. The resulting JSON used by the module would then have the players listed up under the correct country. Taavi (talk!) 18:26, 1 August 2023 (UTC)
- I think the only question left is what to do about Russia and Bulgaria - players from these two countries have blanks in their country field and thus do not get ported over to the json. Is this enough of an issue to hold up the bot task, or is it something that can be worked out over time? Primefac (talk) 13:19, 1 August 2023 (UTC)
Approved. Looks good to me! As per usual, if amendments to - or clarifications regarding - this approval are needed, please start a discussion on the talk page and ping. --TheSandDoctor Talk 16:03, 6 August 2023 (UTC)
- The above discussion is preserved as an archive of the debate. Please do not modify it. To request review of this BRFA, please start a new section at Wikipedia:Bots/Noticeboard.