Merge pull request #2 from carriganm95/update_hitTuning

address PR comments round 1

Author: carriganm95

sbncode/CAFMaker/RecoUtils/CMakeLists.txt

@@ -11,6 +11,4 @@ art_make_library( LIBRARY_NAME caf_RecoUtils
                     larsim::MCCheater_BackTrackerService_service
                     larsim::MCCheater_ParticleInventoryService_service
                     larcorealg::Geometry
-                    sbnanaobj::StandardRecord
-                    sbnanaobj::StandardRecordFlat
                 )

sbncode/HitFinder/GaussHitFinderSBN_module.cc

@@ -71,12 +71,18 @@ namespace hit {
     std::vector<double> FillOutHitParameterVector(const std::vector<double>& input);
 
     template<class T>
-    inline std::vector<T> getValueOrListOf(fhicl::ParameterSet const& pset, std::string const& key) {
+    std::vector<T> getValueOrListOf(fhicl::ParameterSet const& pset, std::string const& key) const {
 
       auto const& wireReadoutGeom = art::ServiceHandle<geo::WireReadout const>()->Get();
       const unsigned int N_PLANES = wireReadoutGeom.Nplanes();
 
       if (pset.is_key_to_sequence(key))
+	auto vec = pset.get<std::vector<T>>(key);
+        if (vec.size() != N_PLANES) {
+            throw cet::exception("Configuration")
+                << "Parameter '" << key << "' has size " << vec.size()
+                << " but expected " << N_PLANES << " (number of planes)";
+        }
         return pset.get<std::vector<T>>(key);
       else
         return std::vector<T>(N_PLANES, pset.get<T>(key));