earth 0.3.11 → 0.3.13

data/lib/earth/air/flight_distance_class/data_miner.rb

@@ -11,5 +11,7 @@ FlightDistanceClass.class_eval do
       key   'name'
       store 'distance', :units_field_name => 'distance_units'
     end
+    
+    # FIXME TODO verify this
   end
 end

data/lib/earth/air/flight_seat_class/data_miner.rb

@@ -3,15 +3,16 @@ FlightSeatClass.class_eval do
     schema Earth.database_options do
       string   'name'
       float    'multiplier'
-      integer  'seats'
     end
     
     import "a list of Brighter Planet-defined seat classes and pre-calculated multipliers",
-           :url => 'http://static.brighterplanet.com/science/data/transport/air/seat_classes/seat_classes.csv' do
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdG5zSER1QmFVVkhNcTM2cmhCMEJtWVE&hl=en&single=true&gid=0&output=csv' do
       key   'name'
       store 'multiplier'
     end
-
+    
+    # FIXME TODO verify this
+    
     # sabshere 5/21/10 in case we ever need this
     # class << self
     #   def refresh
@@ -33,4 +34,3 @@ FlightSeatClass.class_eval do
     # end
   end
 end
-

data/lib/earth/air/flight_segment/data_miner.rb

@@ -314,5 +314,12 @@ FlightSegment.class_eval do
     process "Derive average seats per departure" do
       update_all 'seats = total_seats / departures_performed', 'departures_performed > 0'
     end
+    
+    # FIXME TODO make this verification check actual aircraft codes in Aircraft
+    verify 'All segments have an associated aircraft' do
+      FlightSegment.where(:aircraft_bts_code => [nil, '']).first.nil?
+    end
+    
+    # FIXME TODO finish verification
   end
 end

data/lib/earth/all.rb

@@ -1,6 +1,7 @@
 require 'earth/air'
 require 'earth/automobile'
 require 'earth/bus'
+require 'earth/computation'
 require 'earth/diet'
 require 'earth/fuel'
 require 'earth/hospitality'

data/lib/earth/automobile.rb

@@ -5,4 +5,10 @@ require 'earth/automobile/automobile_make_year'
 require 'earth/automobile/automobile_make_model'
 require 'earth/automobile/automobile_make_model_year'
 require 'earth/automobile/automobile_size_class'
+require 'earth/automobile/automobile_size_class_year'
 require 'earth/automobile/automobile_make_model_year_variant'
+require 'earth/automobile/automobile_type_year'
+require 'earth/automobile/automobile_type_fuel_age'
+require 'earth/automobile/automobile_type_fuel_control'
+require 'earth/automobile/automobile_type_fuel_year'
+require 'earth/automobile/automobile_type_fuel_year_control'

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

@@ -3,54 +3,97 @@ AutomobileFuelType.class_eval do
     schema Earth.database_options do
       string   'code'
       string   'name'
+      string   'fuel_name_for_distance'
+      string   'fuel_name_for_efs'
+      float    'blend_portion'
       float    'emission_factor'
       string   'emission_factor_units'
       float    'annual_distance'
       string   'annual_distance_units'
     end
     
-    import("the fuels used in the 2000 EPA fuel economy guide",
-           :url => 'http://www.fueleconomy.gov/FEG/epadata/00data.zip',
-           :filename => 'Gd6-dsc.txt',
-           :format => :fixed_width,
-           :crop => 21..26, # inclusive
-           :cut => '2-',
-           :select => lambda { |row| /\A[A-Z]/.match row[:code] },
-           :schema => [[ 'code',   2,  { :type => :string }  ],
-                       [ 'spacer', 2 ],
-                       [ 'name',   52, { :type => :string } ]]) do
+    import "a list of fuel type codes and attributes",
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdDlqeU9vQkVkNG1NZXV4WklKTjJkU3c&hl=en&single=true&gid=0&output=csv' do
       key   'code'
       store 'name'
+      store 'fuel_name_for_distance'
+      store 'fuel_name_for_efs'
+      store 'blend_portion'
     end
     
-    import "a pre-calculated emission factor and average annual distance for each fuel",
-           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdDlqeU9vQkVkNG1NZXV4WklKTjJkU3c&hl=en&single=true&gid=0&output=csv' do
-      key   'code'
-      store 'name'
-      store 'annual_distance', :units_field_name => 'annual_distance_units'
-      store 'emission_factor', :units_field_name => 'emission_factor_units'
+    process "Calculate annual distance from AutomobileTypeFuelAge" do
+      AutomobileTypeFuelAge.run_data_miner!
+      ages = AutomobileTypeFuelAge.arel_table
+      types = AutomobileFuelType.arel_table
+      conditional_relation = ages[:fuel_common_name].eq(types[:fuel_name_for_distance])
+      update_all "annual_distance = (#{AutomobileTypeFuelAge.weighted_average_relation(:annual_distance, :weighted_by => :vehicles).where(conditional_relation).to_sql})"
+      update_all "annual_distance_units = 'kilometres'"
     end
     
-    verify "Annual distance should be greater than zero" do
+    process "Calculate emission factor from FuelYear, AutomobileTypeFuelYear, and GreenhouseGas" do
+      FuelYear.run_data_miner!
+      AutomobileTypeFuelYear.run_data_miner!
+      GreenhouseGas.run_data_miner!
+      
+      co2_gwp = GreenhouseGas.find_by_abbreviation("co2").global_warming_potential
+      ch4_gwp = GreenhouseGas.find_by_abbreviation("ch4").global_warming_potential
+      n2o_gwp = GreenhouseGas.find_by_abbreviation("n2o").global_warming_potential
+      
+      latest_year = FuelYear.all.collect{|x| x.year }.max
+      
+      fuel_years = FuelYear.arel_table
+      type_fuel_years = AutomobileTypeFuelYear.arel_table
+      fuel_types = AutomobileFuelType.arel_table
+      
       AutomobileFuelType.all.each do |fuel_type|
-        unless fuel_type.annual_distance > 0.0
-          raise "Invalid annual_distance for AutomobileFuelType #{fuel_type.name}: #{fuel_type.annual_distance} (should be > 0)"
+        current_name = fuel_type.fuel_name_for_efs
+        
+        unless current_name.nil?
+          co2_factor_sql = fuel_years.project(fuel_years[:co2_emission_factor]).where(fuel_years[:fuel_common_name].eq(fuel_types[:fuel_name_for_efs]).and(fuel_years[:year].eq(latest_year))).to_sql
+          ch4_factor_sql = AutomobileTypeFuelYear.weighted_average_relation(:ch4_emission_factor, :weighted_by => :total_travel).where(type_fuel_years[:fuel_common_name].eq(fuel_types[:fuel_name_for_efs]).and(type_fuel_years[:year].eq(latest_year))).to_sql
+          n2o_factor_sql = AutomobileTypeFuelYear.weighted_average_relation(:n2o_emission_factor, :weighted_by => :total_travel).where(type_fuel_years[:fuel_common_name].eq(fuel_types[:fuel_name_for_efs]).and(type_fuel_years[:year].eq(latest_year))).to_sql
+          hfc_factor_sql = AutomobileTypeFuelYear.weighted_average_relation(:hfc_emission_factor, :weighted_by => :total_travel).where(type_fuel_years[:fuel_common_name].eq(fuel_types[:fuel_name_for_efs]).and(type_fuel_years[:year].eq(latest_year))).to_sql
+          
+          connection.execute %{
+            UPDATE automobile_fuel_types
+            SET emission_factor = (((#{co2_factor_sql}) * #{co2_gwp} * (1 - blend_portion)) + ((#{ch4_factor_sql}) * #{ch4_gwp}) + ((#{n2o_factor_sql}) * #{n2o_gwp}) + (#{hfc_factor_sql}))
+            WHERE automobile_fuel_types.fuel_name_for_efs = '#{current_name}'
+          }
+          connection.execute %{
+            UPDATE automobile_fuel_types
+            SET emission_factor_units = 'kilograms_co2e_per_litre'
+            WHERE automobile_fuel_types.fuel_name_for_efs = '#{current_name}'
+          }
         end
       end
     end
     
-    verify "Annual distance units should never be missing" do
+    process "Calculate emission factor for electricity from EgridSubregion and EgridRegion" do
+      EgridSubregion.run_data_miner!
+      EgridRegion.run_data_miner!
+      electricity_ef = (EgridSubregion.find_by_abbreviation("US").electricity_emission_factor / (1 - EgridRegion.find_by_name("US").loss_factor))
+      connection.execute %{
+        UPDATE automobile_fuel_types
+        SET emission_factor = #{electricity_ef}
+        WHERE automobile_fuel_types.name = 'electricity'
+      }
+    end
+    
+    verify "Annual distance and emission factor should be greater than zero" do
       AutomobileFuelType.all.each do |fuel_type|
-        if fuel_type.annual_distance_units.nil?
-          raise "Missing annual distance units for AutomobileFuelType #{fuel_type.name}"
+        %w{ annual_distance emission_factor }.each do |attribute|
+          value = fuel_type.send(:"#{attribute}")
+          unless value > 0
+            raise "Invalid #{attribute} for AutomobileFuelType #{fuel_type.name}: #{value} (should be > 0)"
+          end
         end
       end
     end
     
-    verify "Emission factor should be zero or more" do
+    verify "Annual distance units should be kilometres" do
       AutomobileFuelType.all.each do |fuel_type|
-        unless fuel_type.emission_factor >= 0.0
-          raise "Invalid emission_factor for AutomobileFuelType #{fuel_type.name}: #{fuel_type.emission_factor} (should be >= 0)"
+        unless fuel_type.annual_distance_units == "kilometres"
+          raise "Missing annual distance units for AutomobileFuelType #{fuel_type.name}"
         end
       end
     end

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

@@ -10,10 +10,9 @@ AutomobileMakeFleetYear.class_eval do
       string   'fuel_efficiency_units'
       integer  'volume'
     end
-
-    # CAFE data privately emailed to Andy from Terry Anderson at the DOT/NHTSA
+    
     import "annual corporate average fuel economy data for domestic and imported vehicle fleets from the NHTSA",
-           :url => 'http://static.brighterplanet.com/science/data/transport/automobiles/make_fleet_years/make_fleet_years.csv',
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdEdXWXB6dkVLWkowLXhYSFVUT01sS2c&hl=en&single=true&gid=0&output=csv',
            :errata => 'http://static.brighterplanet.com/science/data/transport/automobiles/make_fleet_years/errata.csv',
            :select => lambda { |row| row['volume'].to_i > 0 } do
       key   'name', :synthesize => lambda { |row| [ row['manufacturer_name'], row['year_content'], row['fleet'][2,2] ].join ' ' }
@@ -25,10 +24,10 @@ AutomobileMakeFleetYear.class_eval do
       store 'volume'
     end
     
-    verify "Year should be between 1978 and 2007" do
+    verify "Year should be from 1978 to 2009" do
       AutomobileMakeFleetYear.all.each do |fleet_year|
-        unless fleet_year.year > 1977 and fleet_year.year < 2008
-          raise "Invalid year for AutomobileMakeFleetYear #{fleet_year.name}: #{fleet_year.year} (should be between 1978 and 2007)"
+        unless fleet_year.year > 1977 and fleet_year.year < 2010
+          raise "Invalid year for AutomobileMakeFleetYear #{fleet_year.name}: #{fleet_year.year} (should be from 1978 to 2009)"
         end
       end
     end

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

@@ -12,7 +12,7 @@ AutomobileMakeModel.class_eval do
       float   'fuel_efficiency_highway'
       string  'fuel_efficiency_highway_units'
     end
-
+    
     process "Derive model names from automobile make model year variants" do
       AutomobileMakeModelYearVariant.run_data_miner!
       connection.execute %{
@@ -23,7 +23,6 @@ AutomobileMakeModel.class_eval do
     
     # TODO not weighted until we get weightings on auto variants
     process "Derive average fuel economy from automobile make model year variants" do
-      AutomobileMakeModelYearVariant.run_data_miner!
       models = AutomobileMakeModel.arel_table
       variants = AutomobileMakeModelYearVariant.arel_table
       conditional_relation = models[:name].eq(variants[:make_model_name])

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

@@ -15,7 +15,7 @@ AutomobileMakeModelYear.class_eval do
       float    'fuel_efficiency_highway'
       string   'fuel_efficiency_highway_units'
     end
-
+    
     process "Derive model year names from automobile make model year variants" do
       AutomobileMakeModelYearVariant.run_data_miner!
       connection.execute %{
@@ -29,7 +29,6 @@ AutomobileMakeModelYear.class_eval do
     # even without volume-weighting, the values are much better.
     # for example, 20km/l for a toyota prius 2006 vs. 13km/l if you use make years
     process "Calculate city and highway fuel efficiency from automobile make model year variants" do
-      AutomobileMakeModelYearVariant.run_data_miner!
       model_years = AutomobileMakeModelYear.arel_table
       variants = AutomobileMakeModelYearVariant.arel_table
       conditional_relation = model_years[:name].eq(variants[:make_model_year_name])
@@ -42,10 +41,10 @@ AutomobileMakeModelYear.class_eval do
       end
     end
     
-    verify "Year should be between 1985 and 2010" do
+    verify "Year should be from 1985 to 2010" do
       AutomobileMakeModelYear.all.each do |model_year|
         unless model_year.year.to_i > 1984 and model_year.year.to_i < 2011
-          raise "Invalid year for AutomobileMakeModelYear #{model_year.name}: #{model_year.year} (should be between 1985 and 2010)"
+          raise "Invalid year for AutomobileMakeModelYear #{model_year.name}: #{model_year.year} (should be from 1985 to 2010)"
         end
       end
     end

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

@@ -9,7 +9,7 @@ AutomobileMakeModelYearVariant.class_eval do
     end
 
     def is_a_porsche?(row)
-      row['make'] and row['make'].upcase == 'PORSCHE'
+      row['make'].to_s.upcase == 'PORSCHE'
     end
 
     def is_not_a_porsche?(row)
@@ -17,19 +17,19 @@ AutomobileMakeModelYearVariant.class_eval do
     end
 
     def is_a_mercedes_benz?(row)
-      row['make'] and row['make'] =~ /MERCEDES/i
+      row['make'].to_s =~ /MERCEDES/i
     end
 
     def is_a_lexus?(row)
-      row['make'] and row['make'].upcase == 'LEXUS'
+      row['make'].to_s.upcase == 'LEXUS'
     end
 
     def is_a_bmw?(row)
-      row['make'] and row['make'].upcase == 'BMW'
+      row['make'].to_s.upcase == 'BMW'
     end
 
     def is_a_ford?(row)
-      row['make'] and row['make'].upcase == 'FORD'
+      row['make'].to_s.upcase == 'FORD'
     end
 
     def is_a_rolls_royce_and_model_contains_bentley?(row)
@@ -37,11 +37,11 @@ AutomobileMakeModelYearVariant.class_eval do
     end
 
     def is_a_bentley?(row)
-      row['make'] and row['make'].upcase == 'BENTLEY'
+      row['make'].to_s.upcase == 'BENTLEY'
     end
 
     def is_a_rolls_royce?(row)
-      row['make'] and row['make'] =~ /ROLLS/i
+      row['make'].to_s =~ /ROLLS/i
     end
 
     def is_a_turbo_brooklands?(row)
@@ -140,7 +140,7 @@ AutomobileMakeModelYearVariant.class_eval do
       bus[:cut] = '13-' if year == 1995
       bus[:schema_name] = :fuel_economy_guide_b
       bus[:select] = lambda do |row|
-        (row['suppress_code'].blank? or row['suppress_code'].to_i == 0) and row['state_code'] == 'F'
+        (row['suppress_code'].blank? or row['suppress_code'].to_f == 0) and row['state_code'] == 'F'
       end
       Slither.define :fuel_economy_guide_b do |d|
         d.rows do |row|
@@ -314,6 +314,8 @@ AutomobileMakeModelYearVariant.class_eval do
     end
     
     # 1985---1997
+    # FIXME TODO one Jaguar in the 1990 FEG has no model name
+    # FIXME TODO 14 records in the 1995 FEG are missing fuel efficiencies
     (85..97).each do |yy|
       filename = (yy == 96) ? "#{yy}MFGUI.ASC" : "#{yy}MFGUI.DAT"
       import("19#{ yy } Fuel Economy Guide",
@@ -382,6 +384,7 @@ AutomobileMakeModelYearVariant.class_eval do
     end
     
     # 2006--2009
+    # FIXME TODO the 2007 FEG import fails: ArgumentError: unknown encoding name - WINDOWS-936
     {
       2006 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/06data.zip', :filename => '2006_FE_Guide_14-Nov-2005_download.csv' },
       2007 => { :url => 'http://www.fueleconomy.gov/FEG/epadata/07data.zip', :filename => '2007_FE_guide_ALL_no_sales_May_01_2007.xls' },
@@ -459,16 +462,17 @@ AutomobileMakeModelYearVariant.class_eval do
       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
     end
     
-    verify "Year should be between 1985 and 2010" do
+    verify "Year should be from 1985 to 2010" do
       AutomobileMakeModelYearVariant.all.each do |variant|
         unless variant.year > 1984 and variant.year < 2011
-          raise "Invalid year for AutomobileMakeModelYearVariant #{variant.row_hash}: #{variant.year} (should be between 1985 and 2010)"
+          raise "Invalid year for AutomobileMakeModelYearVariant #{variant.row_hash}: #{variant.year} (should be from 1985 to 2010)"
         end
       end
     end

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

@@ -54,10 +54,10 @@ AutomobileMakeYear.class_eval do
       }
     end
     
-    verify "Year should be between 1985 and 2010" do
+    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 between 1985 and 2010)"
+          raise "Invalid year for AutomobileMakeYear #{make_year.name}: #{make_year.year} (should be from 1985 to 2010)"
         end
       end
     end

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

@@ -1,37 +1,39 @@
 AutomobileSizeClass.class_eval do
   data_miner do
     schema Earth.database_options do
-      string   'name'
-      string   'emblem'
-      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'
+      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
-
-    import "a list of size classes and pre-calculated annual distances and fuel efficiencies",
-           :url => 'http://static.brighterplanet.com/science/data/transport/automobiles/models_export/automobile_size_class.csv' do
+    
+    import "A list of size classes and pre-calculated fuel efficiencies",
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdHlRUE5IcWlrRENhN0EtUldPTy1rX1E&single=true&gid=0&output=csv' do
       key 'name'
-      store 'annual_distance', :units => :kilometres
-      store 'fuel_efficiency_city', :units => :kilometres_per_litre
-      store 'fuel_efficiency_highway', :units => :kilometres_per_litre
+      store 'type_name'
+      store 'fuel_efficiency_city', :units_field_name => 'fuel_efficiency_city_units'
+      store 'fuel_efficiency_highway', :units_field_name => 'fuel_efficiency_highway_units'
+    end
+    
+    process "Calculate annual distance from AutomobileTypeFuelAge" do
+      AutomobileTypeFuelAge.run_data_miner!
+      ages = AutomobileTypeFuelAge.arel_table
+      classes = AutomobileSizeClass.arel_table
+      conditional_relation = ages[:type_name].eq(classes[:type_name])
+      update_all "annual_distance = (#{AutomobileTypeFuelAge.weighted_average_relation(:annual_distance, :weighted_by => :vehicles).where(conditional_relation).to_sql})"
+      update_all "annual_distance_units = 'kilometres'"
     end
-
-    # Ian 5/27/2010 I'm pretty sure we don't need emblems in middleware
-    # import "",
-    #        :url => 'http://static.brighterplanet.com/science/data/transport/automobiles/models_export/automobile_size_class_emblems.csv' do
-    #   key 'name'
-    #   store 'emblem'
-    # end
     
     import "pre-calculated hybridity multipliers",
-           :url => 'http://static.brighterplanet.com/science/data/transport/automobiles/vehicle_classes/fuel_efficiency_multipliers.csv' do
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdGt2NnhXLXUxNFRJSzczU3BkSHB3enc&hl=en&single=true&gid=0&output=csv' do
       key 'name'
       store 'hybrid_fuel_efficiency_city_multiplier'
       store 'hybrid_fuel_efficiency_highway_multiplier'

data/lib/earth/automobile/automobile_size_class_year.rb

@@ -0,0 +1,7 @@
+class AutomobileSizeClassYear < ActiveRecord::Base
+  set_primary_key :name
+  
+  data_miner do
+    tap "Brighter Planet's sanitized automobile size class year data", Earth.taps_server
+  end
+end

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

@@ -0,0 +1,56 @@
+AutomobileSizeClassYear.class_eval do
+  data_miner do
+    schema Earth.database_options do
+      string  'name'
+      string  'size_class_name'
+      integer 'year'
+      string  'type_name'
+      float   'fuel_efficiency_city'
+      string  'fuel_efficiency_city_units'
+      float   'fuel_efficiency_highway'
+      string  'fuel_efficiency_highway_units'
+    end
+    
+    import "automobile size class year fuel efficiencies from the 2010 EPA Fuel Economy Trends report",
+           :url => 'https://spreadsheets.google.com/pub?key=0AoQJbWqPrREqdDZOLWdDdlZja04xZkJYc3NjeGxQamc&hl=en&single=true&gid=0&output=csv' do
+      key 'name'
+      store 'size_class_name'
+      store 'year'
+      store 'type_name'
+      store 'fuel_efficiency_city', :units_field_name => 'fuel_efficiency_city_units'
+      store 'fuel_efficiency_highway', :units_field_name => 'fuel_efficiency_highway_units'
+    end
+    
+    # FIXME TODO verify that type_name appears in AutomobileTypeFuelAges
+    
+    verify "Year should be from 1975 to 2010" do
+      AutomobileSizeClassYear.all.each do |record|
+        unless record.year > 1974 and record.year < 2011
+          raise "Invalid year for AutomobileSizeClassYear #{record.name}: #{record.year} (should be from 1975 to 2010)"
+        end
+      end
+    end
+    
+    verify "Fuel efficiencies should be greater than zero" do
+      AutomobileSizeClassYear.all.each do |year|
+        %w{ fuel_efficiency_city fuel_efficiency_highway }.each do |attribute|
+          value = year.send(:"#{attribute}")
+          unless value > 0
+            raise "Invalid #{attribute} for AutomobileSizeClassYear #{year.name}: #{value} (should be > 0)"
+          end
+        end
+      end
+    end
+    
+    verify "Fuel efficiency units should be kilometres per litre" do
+      AutomobileSizeClassYear.all.each do |year|
+        %w{ fuel_efficiency_city_units fuel_efficiency_highway_units }.each do |attribute|
+          value = year.send(:"#{attribute}")
+          unless value == "kilometres_per_litre"
+            raise "Invalid #{attribute} for AutomobileSizeClassYear #{year.name}: #{value} (should be kilometres_per_litre)"
+          end
+        end
+      end
+    end
+  end
+end