Wikipedia:Bots/Requests for approval/DannyS712 bot 41


 * The following discussion is an archived debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA. The result of the discussion was

DannyS712 bot 41
Operator:

Time filed: 04:11, Thursday, May 23, 2019 (UTC)

Automatic, Supervised, or Manual: automatic

Programming language(s): AWB

Source code available: AWB

Function overview: Assess unassessed articles that are part of WikiProject Architecture and are tagged as stubs as stub class

Links to relevant discussions (where appropriate): Wikipedia talk:WikiProject Architecture

Edit period(s): One time run

Estimated number of pages affected: 606

Exclusion compliant (Yes/No): No

Already has a bot flag (Yes/No): Yes

Function details: This is a follow-up to Bots/Requests for approval/DannyS712 bot 21, which suggested that a more limited scale would be ideal, and a follow up to Bots/Requests for approval/DannyS712 bot 35, which was approved to do the same for wikiproject SCOTUS for ~150 pages, showing that the logic of the bot is sound.

Discussion
I have a feeling this will get approved pretty easily, but just on the off chance that something's broken let's give it a trial run. Primefac (talk) 23:03, 23 May 2019 (UTC)
 * BotTrialComplete I reran the numbers, and its only around 500 pages total. I didn't see any issues, edits: Thanks, --DannyS712 (talk) 23:25, 23 May 2019 (UTC)
 * BAGAssistanceNeeded are there any issues that I need to take care of? --DannyS712 (talk) 09:14, 26 May 2019 (UTC)
 * It's been three days. Please be patient. Primefac (talk) 14:15, 26 May 2019 (UTC)

The results of the run appear to be inconsistent. These are in minor ways, but we normally expect bot runs that will be in "automatic mode" to be consistent, can you explain why these should be different:
 * Used leading pipe spacing only:
 * 1
 * 2


 * Used leading and trailing pipe spacing, added an empty parameter
 * 3


 * Used leading and trailing pipe spacing
 * 4
 * 5

Thanks, — xaosflux  Talk 19:22, 27 May 2019 (UTC)
 * Here is what I used in terms of find-and-replace:

->
 * and, only if  wasn't already present (meaning that the first rule did nothing,

->
 * I tweaked the regex over time, and adjust to fit a few unique cases, but generally this should account for the differences. --DannyS712 (talk) 19:30, 27 May 2019 (UTC)
 * so sometimes you will add the empty  parameter, and sometimes not? Why be inconsistent here?  Do you expect to have consistent whitespace going forward as well? —  xaosflux  Talk 20:38, 27 May 2019 (UTC)
 * I can tweak it so that it always adds an empty importance parameter (unless there already is an importance set) and move more consistent whitespace, sorry --DannyS712 (talk) 20:46, 27 May 2019 (UTC)
 * it doesn't really matter to me if it is or isn't there, or if there is or isn't extra whitespace - but it should be consistent. The documentation at Template:WikiProject Architecture suggests that it should have both, and there should not be extra whitespace when using an in-line model. I haven't researched how much discussion came up with the example:   but at least there is something to go off of. If the project wants something else that is fine as well.  What do you think? —  xaosflux  Talk 21:07, 27 May 2019 (UTC)
 * sure, I can do that. --DannyS712 (talk) 21:09, 27 May 2019 (UTC)
 * to verify the new consistent settings discussed above. — xaosflux  Talk 22:00, 27 May 2019 (UTC)
 * - I don't know why my AWB is messing this up, but I keep doing 1 too many edits - 51 edits made, didn't see any issues. --DannyS712 (talk) 22:06, 27 May 2019 (UTC)
 * OBOE's are common - normally operator/programmer error :D — xaosflux  Talk 22:52, 27 May 2019 (UTC)
 * trial looks fine. — xaosflux  Talk 22:53, 27 May 2019 (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 WT:BRFA.