Page MenuHomePhabricator

Add a link: link suggested inside red link
Closed, ResolvedPublic

Description

I saw this in cswiki in production on https://cs.wikipedia.org/wiki/Sandrine_Grudaov%C3%A1?veaction=edit&getasktype=link-recommendation&geclickid=123. The suggestion is to link to "Los Angeles", but that suggestion is actually right in the middle of an existing redlink to "Los Angeles Sparks". We don't want to give link suggestions that overlap links, red or blue.

image.png (407×498 px, 64 KB)

Suggestion data from https://cs.wikipedia.org/w/rest.php/growthexperiments/v0/suggestions/addlink/Sandrine_Grudaov%C3%A1 :

{
  "recommendation": {
    "links": [
      {
        "link_text": "UMMC Jekatěrinburg",
        "link_target": "UGMK Jekatěrinburg",
        "match_index": 0,
        "wikitext_offset": 3443,
        "score": 0.8032418489456177,
        "context_before": "05–2007), ",
        "context_after": " (2007-200",
        "link_index": 0
      },
      {
        "link_text": "Los Angeles",
        "link_target": "Los Angeles",
        "match_index": 0,
        "wikitext_offset": 3515,
        "score": 0.6978706121444702,
        "context_before": "08–2010), ",
        "context_after": " Sparks (2",
        "link_index": 1
      },
      {
        "link_text": "Fenerbahçe Istanbul",
        "link_target": "Fenerbahçe SK",
        "match_index": 0,
        "wikitext_offset": 3554,
        "score": 0.5645785927772522,
        "context_before": "6, 2017), ",
        "context_after": " (2016–201",
        "link_index": 2
      }
    ],
    "meta": {
      "application_version": "5b4709d",
      "dataset_checksums": {
        "anchors": "057c4099b9d09607f2b3daaceb248c804d7fc320f20308e54cd2c51fecd411d5",
        "model": "f7b6a47ff5306cb4caea4277545d701534246530caf6a7e30bca313df5cf6c49",
        "pageids": "78cbd8abe7006d0e28c7f796b013e8f8ae0f12473c790a105c0250a3ec08ac2d",
        "redirects": "63e68e83c053351dee982594a57578ae6ea8d15c5e38c76568001349448a1aa7",
        "w2vfiltered": "60a257c5f72fff1667209c2c3fa8bf956111bd2b209970d7d4f2a813039babe5"
      },
      "format_version": 1
    }
  }
}

The article's wikitext:

{{Infobox - osoba}}
{{Infobox - medaile
  | soutěž1 = [[Soubor:Olympic rings.svg|30px|Olympijské kruhy]] [[Basketbal na letních olympijských hrách|Basketbal na LOH]]
  | medaile1 = 
  {{Medaile|2|[[Basketbal na Letních olympijských hrách 2012|LOH 2012]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
  | soutěž2 = [[Mistrovství Evropy v basketbale žen]]
  | medaile2 = 
  {{Medaile|1|[[Mistrovství Evropy v basketbalu žen 2009|ME 2009]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
  {{Medaile|3|[[Mistrovství Evropy v basketbalu žen 2011|ME 2011]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
  {{Medaile|2|[[Mistrovství Evropy v basketbalu žen 2013|ME 2013]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
  {{Medaile|2|[[Mistrovství Evropy v basketbalu žen 2015|ME 2015]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
  {{Medaile|2|[[Mistrovství Evropy v basketbalu žen 2019|ME 2019]]|[[Francouzská ženská basketbalová reprezentace|Francie]]}}
}}
'''Sandrine Grudaová''' (* [[25. červen|25. června]] [[1987]], [[Cannes]]) je [[Francie|francouzská]] [[basketbal]]istka hrající převážně na postu [[Pivot (basketbal)|pivota]]. S [[Francouzská ženská basketbalová reprezentace|francouzskou ženskou basketbalovou reprezentací]] získala stříbrnou medaili na [[Basketbal na Letních olympijských hrách 2012|olympijských hrách v Londýně]] roku 2012<ref>{{Citace elektronického periodika
| titul = Sandrine Gruda Bio, Stats, and Results
| periodikum = Olympics at Sports-Reference.com
| url = http://www.sports-reference.com/olympics/athletes/gr/sandrine-gruda-1.html
| jazyk = en
| datum přístupu = 2019-12-11
| url archivu = https://web.archive.org/web/20121216173627/http://www.sports-reference.com/olympics/athletes/gr/sandrine-gruda-1.html
}}</ref> a vyhrála [[Mistrovství Evropy v basketbalu žen 2009|mistrovství Evropy v roce 2009]]. Krom toho má z [[Mistrovství Evropy v basketbalu žen|Eurobasketu]] tři stříbra (2013, 2015, 2019) a jeden bronz (2011).<ref>{{Citace elektronického periodika
| příjmení =
| jméno =
| titul = Players: Sandrine GRUDA
| periodikum = archive.fiba.com
| vydavatel =
| url = http://archive.fiba.com/pages/eng/fa/p/rpp//q//pid/44236/_//players.html
| datum vydání =
| url archivu =
| datum přístupu = 2019-12-11
}}</ref> Na klubové úrovni patří k jejím největším úspěchům vítězství v severoamerické [[WNBA]] s klubem [[Los Angeles Sparks]] v roce 2016. Stala se tak první Francouzkou, která WNBA vyhrála.<ref>{{Citace elektronické monografie
| příjmení =
| jméno =
| titul = SANDRINE GRUDA LA PREMIÈRE FRANÇAISE À ÊTRE CHAMPIONNE WNBA
| url = http://negronews.fr/sport-sandrine-gruda-la-premiere-francaise-a-etre-championne-wnba/
| vydavatel = NegroNews.fr
| místo =
| datum vydání = 2016-10-24
| datum přístupu = 2019-12-11
| jazyk = fr-FR
}}</ref> K největším jejím individuálním úspěchům pak patří vítězství v anketě [[Mezinárodní basketbalová federace|Mezinárodní basketbalové federace]] (FIBA) o nejlepší basketbalistku Evropy v roce 2009.<ref>{{Citace elektronického periodika
| příjmení =
| jméno =
| titul = Golden Gruda snatches Women's Player of the Year award
| periodikum = FIBA.basketball
| vydavatel =
| url = http://www.fiba.basketball/news/FRA---Golden-Gruda-snatches-Women-s-Player-of-the-Year-award
| datum vydání =
| jazyk = en
| url archivu =
| datum přístupu = 2019-12-11
}}</ref> Působila v klubech US Valenciennes (2005–2007), UMMC Jekatěrinburg (2007-2008, 2010-2014), Connecticut Sun (2008–2010), Los Angeles Sparks (2014, 2016, 2017), Fenerbahçe Istanbul (2016–2017), Yakın Doğu (2018) a PF Schio (2018-dosud).

== Odkazy ==

=== Reference ===
<references />
== Externí odkazy ==
* {{Olympedia}}

{{Olympijské medailistky v basketbalu - turnaj 2012}}
{{Medailistky - Mistrovství Evropy v basketbale 2009}}
{{Medailistky - Mistrovství Evropy v basketbale 2011}}
{{Medailistky - Mistrovství Evropy v basketbale 2013}}
{{Medailistky - Mistrovství Evropy v basketbale 2015}}
{{Medailistky - Mistrovství Evropy v basketbale 2019}}

[[Kategorie:Francouzští basketbalisté]]
[[Kategorie:Mistryně Evropy v basketbalu]]
[[Kategorie:Narození v Cannes]]
[[Kategorie:Narození 1987]]
[[Kategorie:Narození 25. června]]
[[Kategorie:Žijící lidé]]
[[Kategorie:Ženy]]

Event Timeline

This is an instance of T267694: Add a link in VE: define exclusion rules for finding text in the DOM: the recommendation service only takes top-level plain text into account when calculating match offsets, and we don't replicate that behavior on the client side (we take all text into account that's not inside a template). So a link recommendation that's for a later (non-linked) occurrence of "Los Angeles" gets matched to the wrong position.

kostajh triaged this task as Medium priority.May 4 2021, 8:45 AM
kostajh updated the task description. (Show Details)

This is an instance of T267694: Add a link in VE: define exclusion rules for finding text in the DOM: the recommendation service only takes top-level plain text into account when calculating match offsets, and we don't replicate that behavior on the client side (we take all text into account that's not inside a template). So a link recommendation that's for a later (non-linked) occurrence of "Los Angeles" gets matched to the wrong position.

While this is a case of T267694, just noting that the suggestion is still from a product perspective, the second "Los Angeles" is still not a good match because it occurs in the phrase "Los Angeles Sparks" (cc @MGerlach). There should ideally be no suggestion, since "Los Angeles Sparks" doesn't exist on the wiki.

Change 684810 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/GrowthExperiments@master] Exclude mw:WikiLink nodes when annotating suggestions

https://gerrit.wikimedia.org/r/684810

Change 684810 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Exclude mw:WikiLink nodes when annotating suggestions

https://gerrit.wikimedia.org/r/684810

Moved to Test in Production - in betalabs no issues have been found.