earth 0.5.4 → 0.6.0

data/lib/earth/automobile/automobile_make_model_year/data_miner.rb

@@ -9,9 +9,19 @@ AutomobileMakeModelYear.class_eval do
     end
     
     process "Derive model year names from automobile make model year variants" do
-      INSERT_IGNORE %{INTO automobile_make_model_years(name, make_name, model_name, make_model_name, year, make_year_name)
-        SELECT automobile_make_model_year_variants.make_model_year_name, automobile_make_model_year_variants.make_name, automobile_make_model_year_variants.name, automobile_make_model_year_variants.make_model_name, automobile_make_model_year_variants.year, automobile_make_model_year_variants.make_year_name FROM automobile_make_model_year_variants WHERE LENGTH(automobile_make_model_year_variants.make_name) > 0 AND LENGTH(automobile_make_model_year_variants.make_model_name) > 0
-      }
+      ::Earth::Utils.insert_ignore(
+        :src => AutomobileMakeModelYearVariant,
+        :dest => AutomobileMakeModelYear,
+        :cols => {
+          :make_model_year_name => :name,
+          :make_name => :make_name,
+          :name => :model_name,
+          :make_model_name => :make_model_name,
+          :year => :year,
+          :make_year_name => :make_year_name
+        }
+        # :where => 'LENGTH(src.make_name) > 0 AND LENGTH(src.make_model_name) > 0'
+      )
     end
     
     # FIXME TODO make this a method on AutomobileMakeModelYear?
@@ -28,37 +38,7 @@ AutomobileMakeModelYear.class_eval do
         # sabshere 12/6/10 careful, don't use AutomobileMakeModelYearVariant.where here or you will be forced into projecting *
         relation = variants.project(variants[:"fuel_efficiency_#{i}"].average).where(conditional_relation).where(null_check)
         update_all "fuel_efficiency_#{i} = (#{relation.to_sql})"
-        update_all "fuel_efficiency_#{i}_units = 'kilometres_per_litre'"
-      end
-    end
-    
-    verify "Year should be from 1985 to 2011" do
-      AutomobileMakeModelYear.all.each do |model_year|
-        unless model_year.year.to_i > 1984 and model_year.year.to_i < 2012
-          raise "Invalid year for AutomobileMakeModelYear #{model_year.name}: #{model_year.year} (should be from 1985 to 2011)"
-        end
-      end
-    end
-    
-    verify "Fuel efficiencies should be greater than zero" do
-      AutomobileMakeModelYear.all.each do |model_year|
-        %w{ city highway }.each do |type|
-          fuel_efficiency = model_year.send(:"fuel_efficiency_#{type}")
-          unless fuel_efficiency.to_f > 0
-            raise "Invalid fuel efficiency #{type} for AutomobileMakeModelYear #{model_year.name}: #{fuel_efficiency} (should be > 0)"
-          end
-        end
-      end
-    end
-    
-    verify "Fuel efficiency units should be kilometres per litre" do
-      AutomobileMakeModelYear.all.each do |model_year|
-        %w{ city highway }.each do |type|
-          units = model_year.send(:"fuel_efficiency_#{type}_units")
-          unless units == "kilometres_per_litre"
-            raise "Invalid fuel efficiency #{type} units for AutomobileMakeModelYear #{model_year.name}: #{units} (should be kilometres_per_litre)"
-          end
-        end
+        update_all "fuel_efficiency_#{i}_units" => 'kilometres_per_litre'
       end
     end
   end

data/lib/earth/automobile/automobile_make_model_year_variant.rb

@@ -6,41 +6,59 @@ class AutomobileMakeModelYearVariant < ActiveRecord::Base
   belongs_to :make_model_year, :class_name => 'AutomobileMakeModelYear', :foreign_key => 'make_model_year_name'
   belongs_to :fuel,            :class_name => 'AutomobileFuel',          :foreign_key => 'fuel_code', :primary_key => 'code'
   
-  force_schema do
-    string   'row_hash'
-    string   'name' # short name!
-    string   'make_name'
-    string   'make_model_name' # make + model
-    string   'make_year_name' # make + year
-    string   'make_model_year_name' # make + model + year
-    integer  'year'
-    float    'fuel_efficiency'
-    string   'fuel_efficiency_units'
-    float    'fuel_efficiency_city'
-    string   'fuel_efficiency_city_units'
-    float    'fuel_efficiency_highway'
-    string   'fuel_efficiency_highway_units'
-    string   'fuel_code'
-    string   'transmission'
-    string   'drive'
-    boolean  'turbo'
-    boolean  'supercharger'
-    integer  'cylinders'
-    float    'displacement'
-    float    'raw_fuel_efficiency_city'
-    string   'raw_fuel_efficiency_city_units'
-    float    'raw_fuel_efficiency_highway'
-    string   'raw_fuel_efficiency_highway_units'
-    integer  'carline_mfr_code'
-    integer  'vi_mfr_code'
-    integer  'carline_code'
-    integer  'carline_class_code'
-    boolean  'injection'
-    string   'carline_class_name'
-    string   'speeds'
-    index    'make_name'
-    index    'make_model_name'
-    index    'make_year_name'
-    index    'make_model_year_name'
-  end
-end
+  col :row_hash
+  col :name # short name!
+  col :make_name
+  col :make_model_name # make + model
+  col :make_year_name # make + year
+  col :make_model_year_name # make + model + year
+  col :year, :type => :integer
+  col :fuel_efficiency, :type => :float
+  col :fuel_efficiency_units
+  col :fuel_efficiency_city, :type => :float
+  col :fuel_efficiency_city_units
+  col :fuel_efficiency_highway, :type => :float
+  col :fuel_efficiency_highway_units
+  col :fuel_code
+  col :transmission
+  col :drive
+  col :turbo, :type => :boolean
+  col :supercharger, :type => :boolean
+  col :cylinders, :type => :integer
+  col :displacement, :type => :float
+  col :raw_fuel_efficiency_city, :type => :float
+  col :raw_fuel_efficiency_city_units
+  col :raw_fuel_efficiency_highway, :type => :float
+  col :raw_fuel_efficiency_highway_units
+  col :carline_mfr_code, :type => :integer
+  col :vi_mfr_code, :type => :integer
+  col :carline_code, :type => :integer
+  col :carline_class_code, :type => :integer
+  col :injection, :type => :boolean
+  col :carline_class_name
+  col :speeds
+  add_index :make_name
+  add_index :make_model_name
+  add_index :make_year_name
+  add_index :make_model_year_name
+  
+  # verify "Fuel code should appear in AutomobileFuel" do
+  #   if (violators = connection.select_values("SELECT DISTINCT fuel_code FROM #{quoted_table_name} WHERE fuel_code NOT IN (SELECT code FROM #{AutomobileFuel.quoted_table_name})")).any?
+  #     raise "Invalid fuel code(s): #{violators.join(', ')}"
+  #   end
+  #   true
+  # end
+  # 
+  # verify "Fuel efficiencies should be greater than zero" do
+  #   [:fuel_efficiency, :fuel_efficiency_city, :fuel_efficiency_highway].each do |field|
+  #     if AutomobileMakeModelYearVariant.where(field => nil).any?
+  #       raise "Invalid #{field} in automobile_make_model_year_variants: nil is not > 0"
+  #     else
+  #       min = AutomobileMakeModelYearVariant.minimum(field)
+  #       unless min > 0
+  #         raise "Invalid #{field} in automobile_make_model_year_variants: #{min} is not > 0"
+  #       end
+  #     end
+  #   end
+  # end
+end

data/lib/earth/automobile/automobile_make_model_year_variant/data_miner.rb

@@ -314,7 +314,7 @@ AutomobileMakeModelYearVariant.class_eval do
       2003 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/03data.zip', :filename => 'guide_2003_feb04-03b.csv' },
       2004 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/04data.zip', :filename => 'gd04-Feb1804-RelDtFeb20.csv' },
       2005 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/05data.zip', :filename => 'guide2005-2004oct15.csv' }
-    }.sort { |a, b| a.first <=> b.first }.each do |year, options|
+    }.each do |year, options|
       import "#{ year } Fuel Economy Guide",
              options.merge(:transform => { :class => AutomobileMakeModelYearVariant::ParserC, :year => year },
                            :errata => { :url => 'https://spreadsheets.google.com/spreadsheet/pub?key=0AoQJbWqPrREqdEFqVXRvQjRGNUpxNHFCOXhqSjRmdlE&output=csv', :responder => AutomobileMakeModelYearVariant::Guru.new },
@@ -348,7 +348,7 @@ AutomobileMakeModelYearVariant.class_eval do
       2007 => { :url => 'http://static.brighterplanet.com/science/data/transport/automobiles/fuel_economy_guide/2007_FE_guide_ALL_no_sales_May_01_2007.csv' },
       2008 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/08data.zip', :filename => '2008_FE_guide_ALL_rel_dates_-no sales-for DOE-5-1-08.csv' },
       2009 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/09data.zip', :filename => '2009_FE_guide for DOE_ALL-rel dates-no-sales-8-28-08download.csv' },
-    }.sort { |a, b| a.first <=> b.first }.each do |year, options|
+    }.each do |year, options|
       import "#{ year } Fuel Economy Guide",
              options.merge(:transform => { :class => AutomobileMakeModelYearVariant::ParserD, :year => year },
                            :errata => { :url => 'https://spreadsheets.google.com/spreadsheet/pub?key=0AoQJbWqPrREqdEFqVXRvQjRGNUpxNHFCOXhqSjRmdlE&output=csv', :responder => AutomobileMakeModelYearVariant::Guru.new },
@@ -379,7 +379,7 @@ AutomobileMakeModelYearVariant.class_eval do
     {
       2010 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/10data.zip', :filename => '2010FEGuide for DOE-all rel dates-no-sales-02-22-2011public.xlsx' },
       2011 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/11data.zip', :filename => '2011FEGuide-for DOE rel-dates before 1-23-2011-no-sales-01-10-2011_All_public.xlsx' }
-    }.sort { |a, b| a.first <=> b.first }.each do |year, options|
+    }.each do |year, options|
       import "#{ year } Fuel Economy Guide",
              options.merge(:transform => { :class => AutomobileMakeModelYearVariant::ParserE, :year => year },
                            :errata => { :url => 'https://spreadsheets.google.com/spreadsheet/pub?key=0AoQJbWqPrREqdEFqVXRvQjRGNUpxNHFCOXhqSjRmdlE&output=csv', :responder => AutomobileMakeModelYearVariant::Guru.new },
@@ -407,22 +407,16 @@ AutomobileMakeModelYearVariant.class_eval do
     end
     
     process "Derive model and model year names" do
-      if ActiveRecord::Base.connection.adapter_name == 'sqlite'
-        update_all "make_model_name = make_name || ' ' || name"
-        update_all "make_year_name = make_name || ' ' || year"
-        update_all "make_model_year_name = make_name || ' ' || name || ' ' || year"
-      else
-        update_all "make_model_name = CONCAT(make_name, ' ', name)"
-        update_all "make_year_name = CONCAT(make_name, ' ', year)"
-        update_all "make_model_year_name = CONCAT(make_name, ' ', name, ' ', year)"
-      end
+      update_all "make_model_name = make_name || ' ' || name"
+      update_all "make_year_name = make_name || ' ' || year"
+      update_all "make_model_year_name = make_name || ' ' || name || ' ' || year"
     end
     
     # Note: need to divide by 0.425143707 b/c equation is designed for miles / gallon not km / l
     # Note: EPA seems to adjust differently for plug-in hybrid electric vehicles (e.g. Leaf and Volt)
     process "Calculate adjusted fuel efficiency using the latest EPA equations from EPA Fuel Economy Trends report Appendix A including conversion from miles per gallon to kilometres per litre" do
-      update_all 'fuel_efficiency_city = 1 / ((0.003259 / 0.425143707) + (1.1805 / raw_fuel_efficiency_city))'
-      update_all 'fuel_efficiency_highway = 1 / ((0.001376 / 0.425143707) + (1.3466 / raw_fuel_efficiency_highway))'
+      update_all 'fuel_efficiency_city = 1 / ((0.003259 / 0.425143707) + (1.1805 / raw_fuel_efficiency_city))', 'raw_fuel_efficiency_city > 0'
+      update_all 'fuel_efficiency_highway = 1 / ((0.001376 / 0.425143707) + (1.3466 / raw_fuel_efficiency_highway))', 'raw_fuel_efficiency_highway > 0'
     end
     
     # This will be useful later for calculating MakeModel and Make fuel efficiency based on Variant
@@ -430,58 +424,18 @@ AutomobileMakeModelYearVariant.class_eval do
     # This results in a deviation from EPA fuel economy label values which use a historical 55/45 weighting
     process "Calculate combined adjusted fuel efficiency using the latest EPA equation" do
       update_all "fuel_efficiency = 1 / ((0.43 / fuel_efficiency_city) + (0.57 / fuel_efficiency_highway))"
-      update_all "fuel_efficiency_units = 'kilometres_per_litre'"
     end
     
-    # FIXME TODO this step fails if the tables to be synthesized do not already exist
-    %w{ AutomobileMakeModelYear AutomobileMakeModel }.each do |synthetic_resource|
-      process "Synthesize #{synthetic_resource}" do
-        synthetic_resource.constantize.run_data_miner!
-      end
+    process "Set units" do
+      update_all :fuel_efficiency_units => 'kilometres_per_litre'
+      update_all :fuel_efficiency_city_units => 'kilometres_per_litre'
+      update_all :fuel_efficiency_highway_units => 'kilometres_per_litre'
     end
     
-    verify "Year should be from 1985 to 2011" do
-      connection.select_values("SELECT DISTINCT year FROM automobile_make_model_year_variants").each do |year|
-        unless year > 1984 and year < 2012
-          raise "Invalid year in automobile_make_model_year_variants: #{year} is not from 1985 to 2011"
-        end
-      end
-    end
-    
-    process "Ensure AutomobileFuel is populated" do
+    process "Ensure related tables are populated" do
+      AutomobileMakeModelYear.run_data_miner!
+      AutomobileMakeModel.run_data_miner!
       AutomobileFuel.run_data_miner!
     end
-    
-    verify "Fuel code should appear in AutomobileFuel" do
-      valid_codes = connection.select_values("SELECT DISTINCT code FROM automobile_fuels")
-      connection.select_values("SELECT DISTINCT fuel_code FROM automobile_make_model_year_variants").each do |fuel_code|
-        unless valid_codes.include?(fuel_code)
-          raise "Invalide fuel code in automobile_make_model_year_variants: #{fuel_code} is not in #{valid_codes}"
-        end
-      end
-    end
-    
-    verify "Fuel efficiencies should be greater than zero" do
-      [:fuel_efficiency, :fuel_efficiency_city, :fuel_efficiency_highway].each do |field|
-        if AutomobileMakeModelYearVariant.where(field => nil).any?
-          raise "Invalid #{field} in automobile_make_model_year_variants: nil is not > 0"
-        else
-          min = AutomobileMakeModelYearVariant.minimum(field)
-          unless min > 0
-            raise "Invalid #{field} in automobile_make_model_year_variants: #{min} is not > 0"
-          end
-        end
-      end
-    end
-    
-    verify "Fuel efficiency units should be kilometres per litre" do
-      %w{ fuel_efficiency_units fuel_efficiency_city_units fuel_efficiency_highway_units }.each do |field|
-        connection.select_values("SELECT DISTINCT #{field} FROM automobile_make_model_year_variants").each do |value|
-          unless value == 'kilometres_per_litre'
-            raise "Invalid #{field} in automobile_make_model_year_variants: #{value} is not 'kilometres_per_litre'"
-          end
-        end
-      end
-    end
   end
 end

data/lib/earth/automobile/automobile_make_year.rb

@@ -4,12 +4,19 @@ class AutomobileMakeYear < ActiveRecord::Base
   belongs_to :make,      :class_name => 'AutomobileMake',          :foreign_key => 'make_name'
   has_many :fleet_years, :class_name => 'AutomobileMakeFleetYear', :foreign_key => 'make_year_name'
 
-  force_schema do
-    string   'name'
-    string   'make_name'
-    integer  'year'
-    float    'fuel_efficiency'
-    string   'fuel_efficiency_units'
-    integer  'volume' # This will sometimes be null because not all make_years have CAFE data
-  end
+  col :name
+  col :make_name
+  col :year, :type => :integer
+  col :fuel_efficiency, :type => :float
+  col :fuel_efficiency_units
+  col :volume, :type => :integer # This will sometimes be null because not all make_years have CAFE data
+  
+  # FIXME TODO verify make_name and volume?
+  
+  # verify "Fuel efficiency should be greater than zero" do
+  #   if violator = first(:conditions => ['fuel_efficiency IS NULL OR fuel_efficiency <= 0'])
+  #     raise "Invalid fuel efficiency for AutomobileMakeYear #{record.name}: #{record.fuel_efficiency} (should be > 0)"
+  #   end
+  #   true
+  # end
 end

data/lib/earth/automobile/automobile_make_year/data_miner.rb

@@ -10,19 +10,16 @@ AutomobileMakeYear.class_eval do
     end
     
     process "Derive manufacturer names and years from automobile make model year variants" do
-      INSERT_IGNORE %{INTO automobile_make_years(name, make_name, year)
-        SELECT
-          automobile_make_model_year_variants.make_year_name,
-          automobile_make_model_year_variants.make_name,
-          automobile_make_model_year_variants.year
-        FROM automobile_make_model_year_variants
-        WHERE automobile_make_model_year_variants.make_year_name IS NOT NULL
-        AND LENGTH(automobile_make_model_year_variants.make_year_name) > 0
-        AND automobile_make_model_year_variants.make_name IS NOT NULL
-        AND LENGTH(automobile_make_model_year_variants.make_name) > 0
-        AND automobile_make_model_year_variants.year IS NOT NULL
-        AND LENGTH(automobile_make_model_year_variants.year) > 0
-      }
+      ::Earth::Utils.insert_ignore(
+        :src => AutomobileMakeModelYearVariant,
+        :dest => AutomobileMakeYear,
+        :cols => {
+          :make_year_name => :name,
+          :make_name => :make_name,
+          :year => :year
+        }
+        # :where => 'src.make_name IS NOT NULL AND LENGTH(src.make_name) > 0 AND src.year IS NOT NULL AND LENGTH(src.year) > 0'
+      )
     end
     
     # FIXME TODO make this a method on AutomobileMakeYear?
@@ -32,52 +29,21 @@ AutomobileMakeYear.class_eval do
       conditional_relation = make_years[:name].eq(make_fleet_years[:make_year_name])
       relation = AutomobileMakeFleetYear.weighted_average_relation(:fuel_efficiency, :weighted_by => :volume).where(conditional_relation)
       update_all "fuel_efficiency = (#{relation.to_sql})"
-      update_all "fuel_efficiency_units = 'kilometres_per_litre'"
     end
     
     process "Calculate fuel effeciency from automobile make model year variants for makes without CAFE data" do
-      connection.execute %{
-        UPDATE automobile_make_years
-        SET automobile_make_years.fuel_efficiency = (SELECT AVG(automobile_make_model_year_variants.fuel_efficiency) FROM automobile_make_model_year_variants WHERE automobile_make_years.name = automobile_make_model_year_variants.make_year_name)
-        WHERE automobile_make_years.fuel_efficiency IS NULL
-      }
-      connection.execute %{
-        UPDATE automobile_make_years
-        SET automobile_make_years.fuel_efficiency_units = 'kilometres_per_litre'
-        WHERE automobile_make_years.fuel_efficiency_units IS NULL
-      }
+      update_all(
+        %{fuel_efficiency = (SELECT AVG(automobile_make_model_year_variants.fuel_efficiency) FROM automobile_make_model_year_variants WHERE automobile_make_years.name = automobile_make_model_year_variants.make_year_name)},
+        %{fuel_efficiency IS NULL}
+      )
     end
     
     process "Calculate sales volume from automobile make fleet years for makes with CAFE data" do
-      connection.execute %{
-        UPDATE automobile_make_years SET automobile_make_years.volume = (SELECT SUM(automobile_make_fleet_years.volume) FROM automobile_make_fleet_years WHERE automobile_make_fleet_years.make_year_name = automobile_make_years.name)
-      }
-    end
-    
-    # FIXME TODO verify make_name and volume?
-    
-    verify "Year should be from 1985 to 2010" do
-      AutomobileMakeYear.all.each do |make_year|
-        unless make_year.year > 1984 and make_year.year < 2011
-          raise "Invalid year for AutomobileMakeYear #{make_year.name}: #{make_year.year} (should be from 1985 to 2010)"
-        end
-      end
+      update_all %{volume = (SELECT SUM(automobile_make_fleet_years.volume) FROM automobile_make_fleet_years WHERE automobile_make_fleet_years.make_year_name = automobile_make_years.name)}
     end
     
-    verify "Fuel efficiency should be greater than zero" do
-      AutomobileMakeYear.all.each do |make_year|
-        unless make_year.fuel_efficiency > 0
-          raise "Invalid fuel efficiency for AutomobileMakeYear #{make_year.name}: #{make_year.fuel_efficiency} (should be > 0)"
-        end
-      end
-    end
-    
-    verify "Fuel efficiency units should be kilometres per litre" do
-      AutomobileMakeYear.all.each do |make_year|
-        unless make_year.fuel_efficiency_units == "kilometres_per_litre"
-          raise "Invalid fuel efficiency units for AutomobileMakeYear #{make_year.name}: #{make_year.fuel_efficiency_units} (should be kilometres_per_litre)"
-        end
-      end
+    process 'Set units' do
+      update_all "fuel_efficiency_units = 'kilometres_per_litre'"
     end
   end
 end

data/lib/earth/automobile/automobile_size_class.rb

@@ -6,18 +6,82 @@ class AutomobileSizeClass < ActiveRecord::Base
                 :conventional_fuel_efficiency_city_multiplier => 0.987,
                 :conventional_fuel_efficiency_highway_multiplier => 0.996
   
-  force_schema do
-    string 'name'
-    string 'type_name'
-    float  'annual_distance'
-    string 'annual_distance_units'
-    float  'fuel_efficiency_city'
-    string 'fuel_efficiency_city_units'
-    float  'fuel_efficiency_highway'
-    string 'fuel_efficiency_highway_units'
-    float  'hybrid_fuel_efficiency_city_multiplier'
-    float  'hybrid_fuel_efficiency_highway_multiplier'
-    float  'conventional_fuel_efficiency_city_multiplier'
-    float  'conventional_fuel_efficiency_highway_multiplier'
-  end
+  col :name
+  col :type_name
+  col :annual_distance, :type => :float
+  col :annual_distance_units
+  col :fuel_efficiency_city, :type => :float
+  col :fuel_efficiency_city_units
+  col :fuel_efficiency_highway, :type => :float
+  col :fuel_efficiency_highway_units
+  col :hybrid_fuel_efficiency_city_multiplier, :type => :float
+  col :hybrid_fuel_efficiency_highway_multiplier, :type => :float
+  col :conventional_fuel_efficiency_city_multiplier, :type => :float
+  col :conventional_fuel_efficiency_highway_multiplier, :type => :float
+  
+  # FIXME TODO verify type_name?
+  
+  # verify "Annual distance should be greater than zero" do
+  #   AutomobileSizeClass.all.each do |size_class|
+  #     unless size_class.annual_distance > 0
+  #       raise "Invalid annual distance for AutomobileSizeClass #{size_class.name}: #{size_class.annual_distance} (should be > 0)"
+  #     end
+  #   end
+  # end
+  # 
+  # verify "Annual distance units should be kilometres" do
+  #   AutomobileSizeClass.all.each do |size_class|
+  #     unless size_class.annual_distance_units == "kilometres"
+  #       raise "Invalid annual distance units for AutomobileSizeClass #{size_class.name}: #{size_class.annual_distance_units} (should be kilometres)"
+  #     end
+  #   end
+  # end
+  # 
+  #  verify "Fuel efficiencies should be greater than zero" do
+  #   AutomobileSizeClass.all.each do |size_class|
+  #     %w{ city highway }.each do |type|
+  #       fuel_efficiency = size_class.send(:"fuel_efficiency_#{type}")
+  #       unless fuel_efficiency > 0
+  #         raise "Invalid fuel efficiency #{type} for AutomobileSizeClass #{size_class.name}: #{fuel_efficiency} (should be > 0)"
+  #       end
+  #     end
+  #   end
+  # end
+  # 
+  # verify "Fuel efficiency units should be kilometres per litre" do
+  #   AutomobileSizeClass.all.each do |size_class|
+  #     %w{ city highway }.each do |type|
+  #       units = size_class.send(:"fuel_efficiency_#{type}_units")
+  #       unless units == "kilometres_per_litre"
+  #         raise "Invalid fuel efficiency #{type} units for AutomobileSizeClass #{size_class.name}: #{units} (should be kilometres_per_litre)"
+  #       end
+  #     end
+  #   end
+  # end
+  # 
+  # verify "Any fuel efficiency multipliers should be greater than zero" do
+  #   AutomobileSizeClass.all.each do |size_class|
+  #     %w{ hybrid conventional }.each do |hybridity|
+  #       %w{ city highway }.each do |type|
+  #         multiplier = size_class.send(:"#{hybridity}_fuel_efficiency_#{type}_multiplier")
+  #         if multiplier.present?
+  #           unless multiplier > 0
+  #             raise "Invalid #{hybridity} fuel efficiency #{type} multiplier for AutomobileSizeClass #{size_class.name}: #{multiplier} (should be > 0)"
+  #           end
+  #         end
+  #       end
+  #     end
+  #   end
+  # end
+  # 
+  # verify "Fallback fuel efficiency multipliers should be greater than zero" do
+  #   %w{ hybrid conventional }.each do |hybridity|
+  #     %w{ city highway }.each do |type|
+  #       multiplier = AutomobileSizeClass.fallback.send(:"#{hybridity}_fuel_efficiency_#{type}_multiplier")
+  #       unless multiplier > 0
+  #         raise "Invalid AutomobileSizeClass fallback #{hybridity} fuel efficiency #{type} multiplier: #{multiplier} (should be > 0)"
+  #       end
+  #     end
+  #   end
+  # end
 end