I’m not really sure what the cause of the problem is… as far as I can see, you’re passing in the right parameters and range. If there was some mix-up somewhere between radians and degrees or lat/lng around the wrong way (I’ve had bad data imports before which have done that), then you wouldn’t even get the two closest results, I would think.
I guess I’d be debugging by firing up a mysql console to Sphinx (mysql —host 127.0.0.1 —port 9306) and running SphinxQL queries to confirm attribute values (SELECT * FROM location_core WHERE sphinx_internal_id = x). On 23 Jan 2014, at 2:21 am, C Wilson <[email protected]> wrote: > Thanks so much Pat for the response and clarifying exactly what you did. I > understood the code you wrote and I can't believe I wasn't able to piece that > together! > > Quick question: > > :with => {:geodist => 0.0..200_000.0}, > > The 200km is equivalent to 124.274 miles. However for the search it is > pulling from a smaller range than that. It seems like from a 20 mile radius > only. > > As I am doing a search with zip code 30052, however it is only returning two > locations that are near each other. A third location should be appearing also > that's 50 miles away. > > > On Tuesday, January 21, 2014 6:26:26 PM UTC-5, Pat Allan wrote: > So, it seems this is the action that needs changing: > > @latitude = params[:latitude].to_f * Math::PI / 180 > @longitude = params[:longitude].to_f * Math::PI / 180 > @users = Location.search :geo => [@latitude * Math::PI / 180, @longitude > * Math::PI / 180], :with => {:geodist => 0.0..200_000.0}, :order => "geodist > ASC" > > I would switch it to: > > latitude = params[:latitude].to_f * Math::PI / 180 > longitude = params[:longitude].to_f * Math::PI / 180 > > location_ids = Location.search_for_ids( > :geo => [latitude, longitude], > :with => {:geodist => 0.0..200_000.0}, > :order => "geodist ASC” > ) > @users = User.where(:location_id => location_ids) > > Note the following: > * I’ve removed the double radians conversion > * I’m using the search_for_ids method to avoid instantiating unnecessary > location objects. > * Instance variables are only useful for objects used in the views, so I’ve > made a presumption and just set @users, everything else is a local variable > instead. > > If all you want is the user to enter their zip code for the search, then it > would be more like this: > > location = Location.find_by_zipcode params[:zipcode] > # if lat/lng are stored as degrees > latitude = location.latitude * Math::PI / 180 > longitude = location.longitude * Math::PI / 180 > > location_ids = Location.search_for_ids( > :geo => [latitude, longitude], > :with => {:geodist => 0.0..200_000.0}, > :order => "geodist ASC” > ) > @users = User.where(:location_id => location_ids) > > Hope this helps! > > — > Pat > > On 22 Jan 2014, at 6:09 am, C Wilson <[email protected]> wrote: > > > Hey Pat, you assisted me with the radians/degrees issue. I was looking for > > a clearer idea than what Nicholas suggested for the part I needed advice > > on. > > > > I have TS setup to find locations based on latitude/longitude that are near > > one another. This part functions as expected. > > > > The next step that I am not clear on is how to setup so that if I perform a > > location search using a zip code from the Locations table to have it return > > users who are within the specified distance. > > > > I am building a app so users can find nearby users using their zip code. I > > want users to enter any zip code and it will return users within 200km. > > > > I am not sure if there is more required from me in the searches controller > > that will make the search perform how I would like when users a zip code > > within the search box. > > > > > > Location.search :geo => [33.89 * Math::PI / 180, -83.95 * Math::PI / 180], > > :with => {:geodist => 0.0..200_000.0}, :order => "geodist ASC" > > Sphinx Query (338.5ms) SELECT GEODIST(0.5914920835008783, > > -1.4652039070492398, latitude, longitude) AS geodist, * FROM > > `location_core` WHERE `geodist` BETWEEN 0.0 AND 200000.0 AND > > `sphinx_deleted` = 0 ORDER BY `geodist` ASC LIMIT 0, 20 > > Sphinx Found 983 results > > Location Load (0.5ms) SELECT `locations`.* FROM `locations` WHERE > > `locations`.`id` IN (12828, 12848, 12882, 12851, 12852, 12853, 12854, > > 12855, 12859, 12830, 12856, 12857, 12822, 12834, 12836, 12838, 12898, > > 12899, 12900, 12901) > > => [#<Location id: 12828, zipcode: "30017", city: "Grayson", state: "GA", > > latitude: 33.89, longitude: -83.95, created_at: nil, updated_at: nil>, > > #<Location id: 12848, zipcode: "30039", city: "Snellville", state: "GA", > > latitude: 33.85, longitude: -84.0, created_at: nil, updated_at: nil>, > > #<Location id: 12882, zipcode: "30078", city: "Snellville", state: "GA", > > latitude: 33.85, longitude: -84.0, created_at: nil, updated_at: nil>, > > #<Location id: 12851, zipcode: "30042", city: "Lawrenceville", state: "GA", > > latitude: 33.94, longitude: -83.99, created_at: nil, updated_at: nil>, > > #<Location id: 12852, zipcode: "30043", city: "Lawrenceville", state: "GA", > > latitude: 33.94, longitude: -83.99, created_at: nil, updated_at: nil>, > > #<Location id: 12853, zipcode: "30044", city: "Lawrenceville", state: "GA", > > latitude: 33.94, longitude: -83.99, created_at: nil, updated_at: nil>, > > #<Location id: 12854, zipcode: "30045", city: "Lawrenceville", state: "GA", > > latitude: 33.94, longitude: -83.99, created_at: nil, updated_at: nil>, > > #<Location id: 12855, zipcode: "30046", city: "Lawrenceville", state: "GA", > > latitude: 33.94, longitude: -83.99, created_at: nil, updated_at: nil>, > > #<Location id: 12859, zipcode: "30052", city: "Loganville", state: "GA", > > latitude: 33.83, longitude: -83.89, created_at: nil, updated_at: nil>, > > #<Location id: 12830, zipcode: "30019", city: "Dacula", state: "GA", > > latitude: 33.98, longitude: -83.88, created_at: nil, updated_at: nil>, > > #<Location id: 12856, zipcode: "30047", city: "Lilburn", state: "GA", > > latitude: 33.88, longitude: -84.13, created_at: nil, updated_at: nil>, > > #<Location id: 12857, zipcode: "30048", city: "Lilburn", state: "GA", > > latitude: 33.88, longitude: -84.13, created_at: nil, updated_at: nil>, > > #<Location id: 12822, zipcode: "30011", city: "Auburn", state: "GA", > > latitude: 34.01, longitude: -83.83, created_at: nil, updated_at: nil>, > > #<Location id: 12834, zipcode: "30024", city: "Suwanee", state: "GA", > > latitude: 34.05, longitude: -84.07, created_at: nil, updated_at: nil>, > > #<Location id: 12836, zipcode: "30026", city: "North Metro", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>, > > #<Location id: 12838, zipcode: "30029", city: "North Metro", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>, > > #<Location id: 12898, zipcode: "30095", city: "Duluth", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>, > > #<Location id: 12899, zipcode: "30096", city: "Duluth", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>, > > #<Location id: 12900, zipcode: "30097", city: "Duluth", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>, > > #<Location id: 12901, zipcode: "30098", city: "Duluth", state: "GA", > > latitude: 34.0, longitude: -84.15, created_at: nil, updated_at: nil>] > > > > > > user = User.first > > User Load (0.3ms) SELECT `users`.* FROM `users` ORDER BY `users`.`id` > > ASC LIMIT 1 > > => #<User id: 1, email: "[email protected]", password_digest: > > "$2a$10$OuN0/fqWzy4ZTyIf4y94teIdnndAqwS.1V5zxgO9X7wW...", zip_code: > > "30052", birthday: "1986-08-25", name: nil, username: "admin", gender: > > "Male", ethnicity: nil, sexuality: "Straight", career: nil, education: nil, > > religion: nil, politics: nil, children: nil, height: nil, user_smoke: nil, > > user_drink: nil, about_me: nil, inches: nil, feet: nil, created_at: > > "2014-01-16 14:17:36", updated_at: "2014-01-16 14:17:36", auth_token: > > "8Zwh3HF6WnA8wthFZirkQw", password_reset_token: nil, > > password_reset_sent_at: nil, admin: nil, role: "admin", roles_mask: nil, > > age: nil, default_photo_id: nil, location_id: nil, time_zone: nil> > > 2.0.0-p353 :004 > user.location > > (15.4ms) SELECT COUNT(*) FROM `locations` WHERE (zipcode = '30052') > > Location Load (6.8ms) SELECT `locations`.* FROM `locations` WHERE > > (zipcode = '30052') ORDER BY `locations`.`id` ASC LIMIT 1 > > => #<Location id: 12859, zipcode: "30052", city: "Loganville", state: > > "GA", latitude: 33.83, longitude: -83.89, created_at: nil, updated_at: nil> > > > > > > > > Location columns: > > > > t.string :zipcode > > t.string :city > > t.string :state > > t.float :latitude > > t.float :longitude > > > > User columns: > > t.string :email > > t.string :password_digest > > t.string :zip_code > > t.string :location_id > > t.string :time_zone > > t.string :birthday > > t.string :name > > t.string :username > > t.string :gender > > t.string :ethnicity > > t.string :sexuality > > t.string :career > > t.string :education > > t.string :religion > > t.string :politics > > t.string :children > > t.string :height > > t.string :user_smoke > > t.string :user_drink > > t.string :about_me > > t.string :inches > > t.string :feet > > > > > > class DistanceController < ApplicationController > > def search > > @latitude = params[:latitude].to_f * Math::PI / 180 > > @longitude = params[:longitude].to_f * Math::PI / 180 > > @users = Location.search :geo => [@latitude * Math::PI / 180, > > @longitude * Math::PI / 180], :with => {:geodist => 0.0..200_000.0}, :order > > => "geodist ASC" > > end > > > > def users > > @users = User.all > > end > > > > def index > > @users = User.all > > end > > end > > > > > > ThinkingSphinx::Index.define :location, :with => :active_record do > > indexes city > > > > has "RADIANS(locations.latitude)", :as => :latitude, :type => :float > > has "RADIANS(locations.longitude)", :as => :longitude, :type => :float > > end > > ThinkingSphinx::Index.define :user, :with => :active_record do > > indexes name, :as => :user, :sortable => true > > indexes zip_code > > has created_at, updated_at > > has location.id, :as => :location_id > > end > > > > > > <%= form_tag searches_path, method: :get do %> > > <p> > > <%= text_field_tag :search, params[:search] %> > > <%= button_tag "Search", name: nil %> > > </p> > > <% end %> > > > > -- > > You received this message because you are subscribed to the Google Groups > > "Thinking Sphinx" group. > > To unsubscribe from this group and stop receiving emails from it, send an > > email to [email protected]. > > To post to this group, send email to [email protected]. > > Visit this group at http://groups.google.com/group/thinking-sphinx. > > For more options, visit https://groups.google.com/groups/opt_out. > > > -- > You received this message because you are subscribed to the Google Groups > "Thinking Sphinx" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/thinking-sphinx. > For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Thinking Sphinx" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/thinking-sphinx. For more options, visit https://groups.google.com/groups/opt_out.
