Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restrict search by location to visible bounds #36

Open
lklepner opened this issue Apr 18, 2016 · 5 comments
Open

Restrict search by location to visible bounds #36

lklepner opened this issue Apr 18, 2016 · 5 comments

Comments

@lklepner
Copy link
Contributor

Searching for maps by location returns results which are not visible in the current location. It would be more accurate/useful if the results only contained maps which have visible content with the search bounds.

@mgiraldo
Copy link
Contributor

search by location (i think) returns maps whose bounds intersect with the current map interface rectangle. is this not the case? one change could be to show maps whose bounds are contained by the interface. that would reduce the amount of results significantly, though.

@lklepner
Copy link
Contributor Author

lklepner commented Apr 19, 2016

It seems the search by location is returning maps which don't have visible map (cropped image) within the search bounds. Some of the results are matching on the overscan or color guide, not the cropped area of the map.

If you search maps by location using the zip 10522 (Dobbs Ferry) you'll find the map below in the results (and and a number of other maps which don't actually cover Dobbs) which I've been guessing is a result of using the raw bound of the rectified image, not the bound of the cropped area.
http://maps.nypl.org/warper/maps/26772

@lklepner
Copy link
Contributor Author

I suspect the search results are inaccurate since the clipmap_bounds is being saved incorrectly - #45

@mgiraldo
Copy link
Contributor

yeah. @bertspaan is working on tools that surface problematic maps (no/wrong crop, no bbox, etc). i just started working on saving the map rectify and resampling modes.

@lklepner
Copy link
Contributor Author

awesome. i'll keep focusing on UI refinements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants