LArbys / LArCV

Liquid Argon Computer Vision
11 stars 9 forks source link

SimpleFiller bug #38

Closed vgenty closed 8 years ago

vgenty commented 8 years ago

Line 25 of SimpleFiller has GuasSmearingSigma instead of GausSmearingSigma and initializes to -1.0. PSet never complains about extra configuration options so I never caught this, and since the var is initialized PSet didn't complain that I didn't set it. Hope this didn't effect anyone... Will leave this open for a bit for people to see, then I can fix.

https://github.com/LArbys/LArCV/blob/master/app/APICaffe/SimpleFiller.cxx#L25

vgenty commented 8 years ago

fixed @ https://github.com/LArbys/LArCV/commit/6e58f95bf948179c3875379f57878c883555a7e5

alt text