OpenData-NC / columbus-county-nc

OpenRural installation for Columbus County, NC
http://columbusco.openrural.org/
6 stars 1 forks source link

Search on homepage.html only works for ZIPs #63

Open rtburg opened 12 years ago

rtburg commented 12 years ago

Search for "Whiteville" and "Hammond" (as in "Hammond Drive") yields "Location Not Found"

Search that appear to work OK:

sarahdooley commented 12 years ago

[note from meeting: this one is possibly non-trivial but is high priority]

sarahdooley commented 12 years ago

@rebecca-caktus : how many hours would you estimate for this task?

ghost commented 12 years ago

@rtburg I made a few changes to the OpenBlock search implementation to make it a little more friendly. Search is still quirky, so I'll explain how it works and we can decide what is most important to work on. OpenBlock search looks for matches in the following order:

1. Location

2. Place The current OpenBlock search mishandles Places: if Place results are found then either no results are returned, or a 500 error will be thrown. Because we don't have any Places at the moment, I'll comment that code out. But if we want to enable Place search in the future, then some (potentially significant) work will be required.

3. Geocode

4. Special Case The code attempts to look up the query in the SearchSpecialCase objects. This does not seem to be available in the admin at the moment, although I think that is a straightforward change if you would like to make use of this.

5. Zip Code

Some things to note:

At this time I think it would be most helpful to improve the disambiguation page results & improve performance when searching for things like 'main street whiteville'. Please let us know your thoughts!

rtburg commented 12 years ago

Nicely executed and documented.

Let's keep track of how long it took us to do this. At some point very soon I'm going to need to report to the Knight Foundation a list of all the "bug fixes" we've had to work on.

To deal with cases such as "'main street whiteville'" we can add some standing text to the "Location Not Found Page" that says "Here are some things you might want to try:

  1. Try searching for the name of the street, without the name of the town.
  2. Navigate our full list of streets to find news for an individual block. ..."

I'm not familiar at all with SearchSpecialCase objects, so I'll have to educate myself about those before I can reply.

There seems to be great advantage in create a couple of selectively useful synonym objects. Best use for those seems to be dealing with misspellings, because if I understand correctly "whiteville" cannot be a synonym for both Whiteville area and Whiteville Town Limits, correct?

Good to know about NewsItems not being searched using this view. I can't think of a good reason for that, can you?

Let's mark this issue down as "good enough for now" and come back to it later. Thanks again for the nice documentation, Rebecca.

ghost commented 12 years ago

Note for later: The search changes in OpenBlock are in the search-changes branch in our fork, which is currently deployed on staging. When we finalize what we're going to do and/or move onto production, these changes will need to be merged into the openrural branch.

ghost commented 12 years ago

The related OpenBlock search changes are enumerated in a pull request to our branch.