Summarizing Trial
edit- Built list of players using the top 200 rankings as of 10/22/12 from: http://legacy.tennis.com/rankings/rankings_men.aspx
- First test build on just Andy Roddick, it worked.
- First test build on top 200...
- Bug 1: Discovered that I need to input accent marks for the player list, as the API cannot handle redirects: so Adrián Menéndez instead of Adrian Menendez.
- FIXED With the help of Legoktm, was able to utilize pywikipedia's Page.isRedirectPage() and Page.getRedirectTarget() to sidestep this issue.
- Bug 2: Adrian Mannarino's statistics did not calculate correctly
- FIXED Was not calculating the proper stats for tables with incomplete slams in the most recent year, ie only 3 slams for Adrian Mannarino in 2012.
- Bug 3: Alexandr Dolgopolov's table has empty cells in years prior to the current year, which I did not account for.
- EDITED There should be an "A" in every cell that is currently empty. I just manually went in and fixed this. NOW FIXED: I thought that only a few articles had empty cells in the Singles Performance Timeline Table, but it turns out, a lot of them do. So instead of manually changing them all by hand, I just set HawkEyeBot to recognize the empty cells too.
- Bug 4: Aljaž Bedene's article caused some of my regex's to trigger incorrectly
- FIXED Made regex's a bit more robust.
- Bug 5: For players that don't have Wikipedia articles, exception is thrown.
- FIXED Skip players with no articles using Page.exists()
- Bug 6: For players that have locked articles/bot cannot edit, exception is thrown.
- FIXED Skip players using Page.canBeEdited() and Page.botMayEdit()
- I have to apologize for letting the bot run unattended as it made quite a few incorrect edits. I have since reverted all the edits made on that run.
I have withdrawn the BRFA for now, due to my oversight on how to treat "walkover" wins. I hope to be able to find the time to rebuild this bot to properly address walkovers in the future. Sitongpeng (talk) 22:41, 23 October 2012 (UTC)