Project

General

Profile

Bug #16502

problems using feature/dgg_lightprop

Added by Lynn Garren over 2 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
05/10/2017
Due date:
% Done:

100%

Estimated time:
Spent time:
Occurs In:
Experiment:
-
Co-Assignees:
Duration:

Description

We locally merged this branch with the head of develop and ran unit tests for larsoft, dunetpc, uboonecode, icaruscode, argoneutcode, and lariatsoft. (These changes are local only and will not be committed.)

All tests pass except for two dunetpc tests.
  • test_ExponentialChannelNoiseService
  • build_oplib_dune35t

History

#1 Updated by Lynn Garren over 2 years ago

cd $MRB_BUILDDIR/dunetpc/test/OpDet/build_oplib_dune35t.d
lar -c ./test_build_oplib_dune35t.fcl
.....
%MSG
%MSG-i PhotonVisibilityService:  lar 10-May-2017 11:06:47 CDT JobSetup
PhotonVisbilityService initializing
%MSG
%MSG-s ArtException:  SimPhotonCounter:pmtresponse@Construction  10-May-2017 11:06:47 CDT ModuleConstruction
cet::exception caught in art
---- Can't find key BEGIN
  MakeLightAnalysisTree 
---- Can't find key END
%MSG
Art has completed and will exit with status 65.

#2 Updated by Diego Gamez over 2 years ago

  • % Done changed from 0 to 100

#3 Updated by Diego Gamez over 2 years ago

  • % Done changed from 100 to 50

#4 Updated by Diego Gamez over 2 years ago

  • % Done changed from 50 to 100

I think I fixed it (not sure how to run the test to be sure ...). I had to make a minor modification in test_ExponentialChannelNoiseService.cxx to include the new properties for the TPB now present in larproperties.fcl. So now we would need to build with the dunetpc code in feature/gp_forLArSoftDevelop

#5 Updated by Lynn Garren over 2 years ago

Thanks Diego. We will test and hopefully get this into next weeks release!

#6 Updated by Lynn Garren over 2 years ago

Unfortunately build_oplib_dune35t still fails when using dunetpc feature/dgg_lightprop.

#7 Updated by Lynn Garren over 2 years ago

The error:

cd $MRB_BUILD/dunetpc/test/OpDet/build_oplib_dune35t.d
lar -c ./test_build_oplib_dune35t.fcl
......
This warning message is issued up to 10 times.
%MSG-i LArG4:  LArG4:largeant 15-May-2017 10:53:39 CDT run: 1 subRun: 0 event: 1
Geant4 simulated 10 MC particles, we keep 10 .
%MSG
Graphics systems deleted.
Visualization Manager deleting...
%MSG-s ArtException:  PostProcessPath end_path 15-May-2017 10:53:39 CDT  PostProcessEvent
cet::exception caught in art
---- EventProcessorFailure BEGIN
  An exception occurred during current event processing
  ---- EventProcessorFailure BEGIN
    An exception occurred during current event processing
    ---- ScheduleExecutionFailure BEGIN
      ProcessingStopped.

      ---- ServiceNotFound BEGIN
        Service  unable to find requested service with compiler type name 'cheat::BackTracker'.
        cet::exception going through module 
      ---- ServiceNotFound END
      Exception going through path end_path
    ---- ScheduleExecutionFailure END
  ---- EventProcessorFailure END
---- EventProcessorFailure END
%MSG

#8 Updated by Gianluca Petrillo over 2 years ago

The module SimPhotonCounter (larana) was changed to employ the service of cheat::BackTracker. Jobs using that analyser configured without BackTracker service would fail, including the test in DUNE35t.

I have changed many FHiCL configuration files which appear to run SimPhotonCounter adding a BackTracker service configuration. This includes the unit test which was failing, that now succeeds.

Also, it turns out that the use of BackTracker service in SimPhotonCounter is bound to a sanity check. I have changed the analyser to perform that check only if BackTracker service is available; otherwise, a non-fatal error message will be printed on the console, which nobody will look at as usual. This turns the change into non-breaking though.
That is larana:f18fb2b283ad0405e3c1ed3c84cbcaccb57f1f88.

Code has been updated in the existing branches feature/dgg_lightprop in several repositories. No new branch was necessary, since luckily MicroBooNE configurations already included BackTracker and SBND and DUNE already had that feature branch. Branches have contextually been updated to LArSoft 6.35.00.

#9 Updated by Lynn Garren over 2 years ago

  • Status changed from Assigned to Resolved

Thank you Gianluca. All tests now pass and we can finally merge feature/dgg_lightprop into a release.



Also available in: Atom PDF