pulibrary / pul-store

PUL Administrative Hydra Head for LAE Project
2 stars 1 forks source link

Make Solr Export Blacklight Maps Compatible #328

Closed kevinreiss closed 9 years ago

kevinreiss commented 9 years ago

The blacklight maps readme recommends one of two fields be present in the solr index place_bbox or placename_coords see https://github.com/sul-dlss/blacklight-maps/blob/master/README.md#usage. Does it make more sense to include these as part of the export to solr feature from the respository rather than require the indexing application to resolve the geo-names reference?

jpstroop commented 9 years ago

Because our data is at the country level, the boxes aren't that useful...they overlap a lot of other countries, etc. I left in the ISO codes, hoping whatever mapping lib we use might be able to highlight countries based on those.

-Js

Sent via mobile. Please excuse typos, brevity, etc. On Dec 1, 2014 11:03 PM, "Kevin Reiss" notifications@github.com wrote:

The blacklight maps readme recommends one of two fields be present in the solr index place_bbox or placename_coords see https://github.com/sul-dlss/blacklight-maps/blob/master/README.md#usage. Does it make more sense to include these as part of the export to solr feature from the respository rather than require the indexing application to resolve the geo-names reference?

— Reply to this email directly or view it on GitHub https://github.com/pulibrary/pul-store/issues/328.

kevinreiss commented 9 years ago

How about the placename_coords though? Those should be available via geonames.

jpstroop commented 9 years ago

just as boxes, not polygons

kevinreiss commented 9 years ago

Not relevant for project.