git-fit 0.18.1 → 0.18.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a0433b30c8c6ae304f3cca0d0bdc83d32579566265383eff605e8cee4b4db0f9
-  data.tar.gz: 88056dd48a9c6bfc81853d72c61d1783816921ff546875321de26d2f2241c154
+  metadata.gz: 4885ddbb01fd4680fac6ae16c6fd3c1b90092910abcef29b317cacd351ee3c1a
+  data.tar.gz: e5257d8d6ddcbd46e8e8e07fda87316fcea76ebc1f47ed0da784ba744e6f4e35
 SHA512:
-  metadata.gz: c07db6a65c33503843c0c39ee17affff523ba946f9bc6993dceacbce4e926f2358bc7424bb2e892ff3732e4848f66a7a9bddd9a78f5ad54f49605cd75d987916
-  data.tar.gz: dd0fffe383ce5a329d52a49f6225d325768fcf59a0f3752e27edb460a81e0913bf1a42615c2eea9ddecae723d82f8edbb7657535e7a4d31485e0929728e59671
+  metadata.gz: c398fbe3e1740edf11ee63e02dd7f22816d6c0304144844f66e6a41e42415f79e263a6a8d6b8d26b4959a64c6d38a4e7e971d2bc97465daacc3475a611ebd380
+  data.tar.gz: 3cf2651b1d8754e28f23f682970757b4e4ba3f1cecda3126c127ccb71ebe09f0c9ce2244ea6f48a1119b46d97a110ed15fcc59c0a8f403c8960fc67e6c745b50

data/lib/git_fit/elevation/terrain_coeff.rb

@@ -4,6 +4,8 @@ require 'msgpack'
 require 'net/http'
 require 'uri'
 
+require_relative 'terrain_lookup'
+
 module GitFit
   module Elevation
     module TerrainCoeff
@@ -49,33 +51,39 @@ module GitFit
         end
 
         def hysteresis_threshold(m_val)
-          return 2.0 if m_val <= 0
-          pav = model['pav_h']
-          return 2.0 if m_val <= pav[0][0]
-          return pav[-1][1] if m_val >= pav[-1][0]
-          pav.each_with_index do |(_xk, yk), i|
-            next unless i + 1 < pav.size
-            return yk if m_val <= pav[i + 1][0]
-          end
-          pav[-1][1]
+          Lookup.hysteresis_threshold(model['pav_h'], m_val)
         end
 
         def dem_spacing(gain_per_km:, range_per_km:, seg_density:)
-          nearest_neighbor_spacing(gain_per_km, range_per_km, seg_density)
+          Lookup.nearest_neighbor_spacing(lookup, gain_per_km, range_per_km, seg_density)
         rescue StandardError => e
           warn "TerrainCoeff: dem_spacing fallback to 181m (#{e.message})"
           181
         end
 
         def elevations_for(locations)
+          uniq_vals = uniquify_locations(locations)
+          elevs = {}
+          uniq_vals.each_slice(100).with_index do |slice, bi|
+            elevs.merge!(fetch_dem_batch(slice, bi))
+            sleep 0.15
+          end
+          locations.map { |lat, lng| elevs[grid_key(lat, lng)] }
+        end
+
+        def uniquify_locations(locations)
           uniq = {}
           locations.each { |lat, lng| uniq[grid_key(lat, lng)] ||= [lat, lng] }
-          uniq_vals = uniq.values
+          uniq.values
+        end
 
+        def fetch_dem_batch(slice, batch_idx)
           elevs = {}
-          uniq_vals.each_slice(100).with_index do |slice, bi|
-            qs = slice.map { |la, ln| "#{la},#{ln}" }.join('|')
-            uri = URI("#{DEM_URL}?locations=#{qs.gsub(',', '%2C').gsub('|', '%7C')}")
+          qs = slice.map { |la, ln| "#{la},#{ln}" }.join('|')
+          uri = URI("#{DEM_URL}?locations=#{qs.gsub(',', '%2C').gsub('|', '%7C')}")
+          attempt = 0
+          loop do
+            # rubocop:disable Style/RedundantBegin
             begin
               res = Net::HTTP.get_response(uri)
               if res.code == '200'
@@ -83,16 +91,28 @@ module GitFit
                 slice.each_with_index do |(lat, lng), i|
                   elevs[grid_key(lat, lng)] = results[i]['elevation'] if results[i]
                 end
+                break
+              elsif res.code == '429'
+                attempt += 1
+                if attempt < 5
+                  delay = 2**attempt
+                  warn "DEM batch #{batch_idx}: rate limited, retrying in #{delay}s (attempt #{attempt})"
+                  sleep(delay)
+                else
+                  warn "DEM batch #{batch_idx}: rate limited, giving up after #{attempt} attempts"
+                  break
+                end
               else
-                warn "DEM batch #{bi} HTTP #{res.code}"
+                warn "DEM batch #{batch_idx} HTTP #{res.code}"
+                break
               end
             rescue StandardError => e
-              warn "DEM batch #{bi}: #{e.message}"
+              warn "DEM batch #{batch_idx}: #{e.message}"
+              break
             end
-            sleep 0.15
+            # rubocop:enable Style/RedundantBegin
           end
-
-          locations.map { |lat, lng| elevs[grid_key(lat, lng)] }
+          elevs
         end
 
         def grid_key(lat, lng)
@@ -102,24 +122,6 @@ module GitFit
         def lookup
           model['lookup']
         end
-
-        private
-
-        def nearest_neighbor_spacing(gain_per_km, range_per_km, seg_density)
-          best = nil
-          best_dist = Float::INFINITY
-          lookup.each do |ref|
-            d = Math.sqrt(
-              (gain_per_km.to_f - ref['gk'])**2 +
-              (range_per_km.to_f - ref['rk'])**2 +
-              (seg_density.to_f - ref['sd'])**2,
-            )
-            next unless d < best_dist
-            best_dist = d
-            best = ref
-          end
-          best ? best['sp'] : 181
-        end
       end
     end
   end

data/lib/git_fit/elevation/terrain_lookup.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module GitFit
+  module Elevation
+    module TerrainCoeff
+      module Lookup
+        class << self
+          def nearest_neighbor_spacing(model_lookup, gain_per_km, range_per_km, seg_density)
+            best = nil
+            best_dist = Float::INFINITY
+            model_lookup.each do |ref|
+              d = Math.sqrt(
+                (gain_per_km.to_f - ref['gk'])**2 +
+                (range_per_km.to_f - ref['rk'])**2 +
+                (seg_density.to_f - ref['sd'])**2,
+              )
+              next unless d < best_dist
+              best_dist = d
+              best = ref
+            end
+            best ? best['sp'] : 181
+          end
+
+          def hysteresis_threshold(model_pav_h, m_val)
+            return 2.0 if m_val <= 0
+            return 2.0 if m_val <= model_pav_h[0][0]
+            return model_pav_h[-1][1] if m_val >= model_pav_h[-1][0]
+            model_pav_h.each_with_index do |(_xk, yk), i|
+              next unless i + 1 < model_pav_h.size
+              return yk if m_val <= model_pav_h[i + 1][0]
+            end
+            model_pav_h[-1][1]
+          end
+        end
+      end
+    end
+  end
+end

data/lib/git_fit/elevation/train/collector.rb

@@ -37,7 +37,7 @@ module GitFit
 
         def resolve_sources(names)
           names.map do |name|
-            Sync::Base.adapters.find { |a| a.name.demodulize.downcase == name.to_s.downcase }
+            Sync::Base.adapters.find { |a| a.config_key == name.to_s }
           end.compact
         end
 

data/lib/git_fit/sync/base.rb

@@ -155,7 +155,7 @@ module GitFit
           lat = point['positionLat'] || point['latitude'] || point['lat']
           lng = point['positionLong'] || point['longitude'] || point['lng']
           return nil unless lat && lng
-          [lng.to_f, lat.to_f]
+          [lat.to_f, lng.to_f]
         end
 
         def cumulative_distances(locations)

data/lib/git_fit/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GitFit
-  VERSION = '0.18.1'
+  VERSION = '0.18.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-fit
 version: !ruby/object:Gem::Version
-  version: 0.18.1
+  version: 0.18.3
 platform: ruby
 authors:
 - Lax
@@ -296,6 +296,7 @@ files:
 - lib/git_fit/elevation/data/dem_coeff.json
 - lib/git_fit/elevation/data/dem_lookup.json
 - lib/git_fit/elevation/terrain_coeff.rb
+- lib/git_fit/elevation/terrain_lookup.rb
 - lib/git_fit/elevation/train/collector.rb
 - lib/git_fit/elevation/train/fitter.rb
 - lib/git_fit/elevation/train/runner.rb