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 Bots/Noticeboard. The result of the discussion was

MajavahBot 5
Operator:

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): Bot_requests

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
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)
 * – SD0001  (talk) 12:08, 31 July 2023 (UTC)
 * 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)

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. -- The SandDoctor 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 Bots/Noticeboard.