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

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 23 2017, 9:39 PM

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 closed this task as Resolved.Jan 17 2020, 9:05 AM
Lokal_Profil claimed this task.