Page MenuHomePhabricator

Use first result value when multiple locations exist on a Wikidata item
Closed, ResolvedPublic2 Estimated Story Points

Description

To solve the issues with errors on multiple location values we should:

Event Timeline

Change 785303 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/services/kartotherian@master] Only use first wikidata query result for the same item

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

WMDE-Fisch set the point value for this task to 2.
WMDE-Fisch moved this task from Doing to Tech Review on the WMDE-TechWish-Sprint-2022-04-13 board.
WMDE-Fisch moved this task from Ready for pickup to In sprint on the WMDE-GeoInfo-FocusArea board.

To test this you could use

<mapframe latitude="41.671111" longitude="13.487222" zoom="10" width="400" height="400" align="center">
{
  "type": "ExternalData",
  "service": "geopoint",
  "ids": "Q833129"
}
</mapframe>

Q833129 currently has two locations ^^'

Change 785303 merged by jenkins-bot:

[mediawiki/services/kartotherian@master] Only use first wikidata query result for the same item

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

This is a great first step, but we might want to support the multiple coordinates by rendering them all. @Lucas_Werkmeister_WMDE shared a query which finds valid use cases, here. For example, the first result is a lake with portions in two countries, and coordinates are given for the center of each portion.

Should this ticket be declined since we decided to show all values instead?

As far as I understand this ticket it was about improving the situation from "the code can't deal with multiple coordinates, gives up and the user sees nothing" to "show something, whatever it is". We did that. It's ready to be demoed.

I suggest to make the next step an incremental one that's as simple as the previous one: Just show all coordinates. Everything else (e.g. making the duplicates distinguishable, adding labels, possibly show error messages) can be explored and discussed after this is done.

@thiemowmde Ok thanks for the clarification. Maybe the title just needs a re-naming then to reflect what the task ended up being about. I won't decline it :)

I suggest to make the next step an incremental one that's as simple as the previous one: Just show all coordinates. Everything else (e.g. making the duplicates distinguishable, adding labels, possibly show error messages) can be explored and discussed after this is done.

Makes sense to me.