Page MenuHomePhabricator

Reasearch new rules for Underlinked-tagging
Open, LowPublic

Description

In order to avoid tagging https://en.wikipedia.org/w/index.php?title=Louise_Clough&diff=680666170&oldid=646685728 which is only 2 small sentaces, and has 2 links we should research for a new set of rules or something (perhaps make it more restrictive).

Event Timeline

Josve05a created this task.Sep 12 2015, 2:51 PM
Josve05a updated the task description. (Show Details)
Josve05a raised the priority of this task from to Needs Triage.
Josve05a added a subscriber: Josve05a.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 12 2015, 2:51 PM
Josve05a renamed this task from Reachers new rules for Underlinked-tagging to Reasearch new rules for Underlinked-tagging.Sep 12 2015, 2:54 PM
Josve05a set Security to None.

One idea might be to not tag stub-articles with this template if two or more links is present.

Restricted Application added a subscriber: StudiesWorld. · View Herald TranscriptJan 20 2016, 1:26 AM

Inside the file: autowikibrowser-code\AWB\WikiFunctions\Parse\Tagger.cs (Line no's: ~155-189). Probably just changing the boolean rules around "bool underlinked" to instead factor in total page word count.

Zoranzoki21 triaged this task as Low priority.Aug 24 2017, 3:00 AM
Zoranzoki21 added a subscriber: Zoranzoki21.
Shaded0 added a subscriber: Reedy.Feb 21 2018, 8:34 PM

@Reedy - I wanted to create a Pull Request or branch for this for approval/merge. How do I do this in Phabricator? Is there a doc somewhere on the Phabricator developer workflow? I've only used Git and not SVN so I'd need to do a bit of reading on this also.

Reedy added a comment.Feb 22 2018, 8:06 AM

@Reedy - I wanted to create a Pull Request or branch for this for approval/merge. How do I do this in Phabricator? Is there a doc somewhere on the Phabricator developer workflow? I've only used Git and not SVN so I'd need to do a bit of reading on this also.

The code isn't hosted in Phabricator

You can make a patch against the github repo at https://github.com/reedy/AutoWikiBrowser if that's easier, and then attach a patch to this bug. I'll then commit it to SVN

Sweet, thanks!

I added a semi-arbitrary value since word count isn't the same as the variable "length" here - but here's what I added.

bool underlinked = ((wikiLinkCount < 0.0025 * length) && (length > 800));

Here's the pull Request Link that I added:
https://github.com/reedy/AutoWikiBrowser/pull/3

@Reedy, Gentle reminder. :)