@Bodhisattwa This is the basic info I could glean. Please fill in the rest.

@JeanFred For info all of these are on Wikidata but there doesn't seem to be a good select to get them all so falling back on the Commons lists

In T286354#7202005, @Lokal_Profil wrote:

@Bodhisattwa This is the basic info I could glean. Please fill in the rest.

done, please check if that is ok.

Change 711018 had a related patch set uploaded (by Lokal Profil; author: Lokal Profil):

[labs/tools/heritage@master] [WIP]Add Indian lists on Commons

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

Hi @Bodhisattwa sorry that the delay i flagged ended up being even longer than expected.

Made a first stab at a mapping and I spotted that the documentation of Template:India_heritage_list_row is fairly outdated. Seems like many of the parameters in actual use are undocumented and that some of the documented ones are no longer supported by the template.

Aliases for parameters seem to be quite frequently used which I'm not sure the harvester supports (will dig into it unless @JeanFred remembers?)

In T286354#7271065, @Lokal_Profil wrote:

Hi @Bodhisattwa sorry that the delay i flagged ended up being even longer than expected.

Made a first stab at a mapping and I spotted that the documentation of Template:India_heritage_list_row is fairly outdated. Seems like many of the parameters in actual use are undocumented and that some of the documented ones are no longer supported by the template.

I have updated the documentation page.

Aliases for parameters seem to be quite frequently used which I'm not sure the harvester supports (will dig into it unless @JeanFred remembers?)

I have removed the aliases parameter.

In T286354#7271065, @Lokal_Profil wrote:

Aliases for parameters seem to be quite frequently used which I'm not sure the harvester supports (will dig into it unless @JeanFred remembers?)

I’m not 100% sure, but I think that aliases are not supported.

In T286354#7272423, @Bodhisattwa wrote:

In T286354#7271065, @Lokal_Profil wrote:

Hi @Bodhisattwa sorry that the delay i flagged ended up being even longer than expected.

Made a first stab at a mapping and I spotted that the documentation of Template:India_heritage_list_row is fairly outdated. Seems like many of the parameters in actual use are undocumented and that some of the documented ones are no longer supported by the template.

I have updated the documentation page.

Thanks!

So I think I could get a harvester up and running, it has the following issues though:

1. Since listeria returns the coordinates as a single {{Inline coordinates}} template the harvester cannot separate latitude and longitude therefore cannot interpret the coordinate. There is probably a workaround for this if we edit every sparql call to (also?) return latitude and longitude explicitly (would not have to be displayed). There might be a SQL clean-up step we could do but it's unclear to me which functions are available to us (CHARINDEX isn't in the docker setup at least)
2. It's unclear to me how the p131 and district parameters are meant to interact. One of these should be mapped to the adm2 property.
3. State ISO is currently not included. Could probably be fixed by adding it to the header template.
4. Two lists seem to have some coding issues, see name, think that is a Listeria bug though which might have been fixed in V2.

I don't think any of the above are blockers though (@JeanFred give a shout if you disagree)

It's unclear to me how the p131 and district parameters are meant to interact. One of these should be mapped to the adm2 property.

For context, p131 is expected to be the lowest tier of administrative unit like municipal wards for city-based monuments (e.g. this one) or community development blocks/mandals etc. for rural monuments (e.g. this one). Unfortunately, we could not have reached that level of detail for all Indian monuments on Wikidata. We have fairly consistent data on districts though, which is the second-level administrative division in India, below the level of states or union territories.

Thanks.

I'll leave out p131 for now then.

I've marked the patch as ready for review (@JeanFred).

This is the quickest implementation I could think of. A more in-depth implementation would be to investigate the sparql queries underlying each of the lists then see if they could be boild down to a handfull of distinct queries and have a config set up for each of these.

A further not is that the en.wp lists will still go on being harvested in parallel I don't believe these should clash and images claimed by more than one dataset isn't new.

Change 711018 merged by jenkins-bot:

[labs/tools/heritage@master] Add Indian lists on Commons

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

Mentioned in SAL (#wikimedia-cloud) [2021-08-26T21:26:57Z] <wm-bot> <lokal-profil> Deploy latest from Git master: f8959e7, 96c6f56, 8f46ca8, 841e8e8, 7d3cfbc, 685e4f6, b1a510f, 78b1ce7, 7172b59, a6d1889, 2bfb4a0, a200b30, 62ea8c3, 2522867, 2790623, b4eed97, 3b29047, 1f1d2da, 27c3613, 6003fc6, 6bd1a0d, d577a84 (T278918), 402b161, 2bdd929, 7f50cb3 (T286354), 2be29f1

In T286354#7316653, @Lokal_Profil wrote:

I'll open up a task for that.

That would be helpful.

But unless there is anything else @Bodhisattwa I believe this task can be closed

Sure. Thanks for working on it.