Issue 324 bboxrelevancy #765
Conversation
solr/conf/schema.xml
Outdated
| </schema> | ||
|
|
||
| <!-- for bbox value --> | ||
| <copyField source="*_geom" dest="*_bboxtype"/> |
There was a problem hiding this comment.
Can we assume that all *_geom fields can be copied into _*bboxtype? If I provide point data here, what happens?
There was a problem hiding this comment.
Yeah, I don't know... potentially "bad things". Instead of the wildcard copy, what specific field should be copied?
There was a problem hiding this comment.
I will try to test that out since that is something we should evaluate. If this copying over doesn't work for point fields, don't know if there is such a thing as a conditional copy possible in schema (i.e. If solr_geom is bbox-compatible, copy over)
There was a problem hiding this comment.
Out current schema uses the "ENVELOPE WKT" syntax in the solr_geom field which will copy over fine. Instead of copying all _geom fields, how about just solr_geom?
There was a problem hiding this comment.
@hudajkhan @mejackreed @eliotjordan We are all on the same page here. Let's limit the copy to just the solr_geom field. Want to push that commit @hudajkhan?
There was a problem hiding this comment.
Will make this change now.
ewlarson
force-pushed
the
issue-324-bboxrelevancy
branch
2 times, most recently
from
a790c31
to
88b7ade
Compare
…able via the presence of Settings.OVERLAP_RATIO_BOOST. When a bbox param is present, the Settings.OVERLAP_RATIO_BOOST value will boost results based on the overlap ratio of the result bboxes. Includes a modification to our Solr schema to include a _bboxtype fieldtype. Requires Solr 7.4+ Co-Authored-By: hudajkhan <hudajkhan@users.noreply.github.com>
ewlarson
force-pushed
the
issue-324-bboxrelevancy
branch
from
88b7ade
to
1d9ff42
Compare
This PR adds optional spatial search overlapRatio relevancy support. Configurable via the presence of Settings.OVERLAP_RATIO_BOOST.
When a bbox param is present, the Settings.OVERLAP_RATIO_BOOST value will boost results based on the overlap ratio of the result bboxes.
Includes a modification to our Solr schema to include a _bboxtype fieldtype. Requires Solr 7.4+
Thanks to:
@hudajkhan / Feature development
@ewlarson / Rspec tests