Page MenuHomePhabricator

Better unused images output for sparql harvests
Closed, ResolvedPublic

Description

With https://gerrit.wikimedia.org/r/#/c/379141/ unused images will be sectioned by source page then ordered by monument id.

For sparql harvests this makes less sense since there will (mostly) be one source per id (i.e. one Wikidata object per id).

For these it might make more sense to skip the sections but still make a new gallery for each id. I.e.

<gallery>
File:Foo|[[d:Q123|1]]
File:Foo|[[d:Q123|1]]
</gallery>
<gallery>
File:Foo|[[d:Q231|2]]
File:Foo|[[d:Q231|2]]
</gallery>

rather than

==[[d:Q123]]==
<gallery>
File:Foo|1
File:Foo|1
</gallery>
==[[d:Q231]]==
<gallery>
File:Foo|2
File:Foo|2
</gallery>

Event Timeline

Primarily created this to discuss which is best. There is a TODO in the code which should be removed if we decide to not implement a different format.

Another good change would be to append #P18 to the links. That way it's easy to see if someone has already added an image.

Change 533756 had a related patch set uploaded (by Lokal Profil; owner: Lokal Profil):
[labs/tools/heritage@master] Better formatting of unused_monument report for sparql datasets

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

Another good change would be to append #P18 to the links. That way it's easy to see if someone has already added an image.

After implementing it I realised that this was a stupid idea.. the whole point is that the items have no images. Hence the anchor does not exist. That is why the anchor bit is not part of the patch.

Change 533756 merged by jenkins-bot:
[labs/tools/heritage@master] Better formatting of unused_monument report for sparql datasets

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

Lokal_Profil claimed this task.