hi dominique and other developers of muoncombinedtrackmon, 

muoneffimontool configures trigdecisiontool even if trigger is 
off. this incorrect configuration produces errors in no trigger jobs.
https://atlasrtt.cern.ch/rtt/rel_3/16.6.1.y-val/build/i686-slc5-gcc43-opt/offline/recjobtransformtests/amitag_q121_notrig/esdtoaod.log
+verbatim+
hltconfigsvc                                      warning if you need the hlt configuration, you need to specify a hlt menu xml file (or rely on another source)
hltconfigsvc                                      warning could not initialize the storagemgr
nameauditor                                          info just exited hltconfigsvc with auditor trigger initialize
servicemanager                                      error unable to initialize service &quot;hltconfigsvc&quot;
trigconfigsvc.hltconfigsvc                          error servicelocatorhelper::service: can not locate service trigconf::hltconfigsvc/hltconfigsvc
trigconfigsvc                                     warning failed to retrieve hlt trigconfigsvc: hltconfigsvc = servicehandle('trigconf::hltconfigsvc/hltconfigsvc')
nameauditor                                          info just exited trigconfigsvc with auditor trigger initialize
trigconfigsvc                                       error trigconfigsvc can not return a hltchainlist object from any of the configured sources ds xmll1
trigconfigsvc                                       error trigconfigsvc can not return a ctpconfig object from any of the configured sources ds xmll1
nameauditor                                          info about to enter toolsvc.trig::trigdecisiontool.navigation with auditor trigger initialize
nameauditor                                          info about to enter trigserializecnvsvc with auditor trigger initialize
trigserializecnvsvc                                  info initialize()
nameauditor                                          info just exited trigserializecnvsvc with auditor trigger initialize
nameauditor                                          info just exited toolsvc.trig::trigdecisiontool.navigation with auditor trigger initialize
toolsvc.trig::trigdecisiontool                       info initialized tdt
nameauditor                                          info just exited toolsvc.trig::trigdecisiontool with auditor trigger initialize
toolsvc.muoneffimon                                  info trigdecisiontool retrieved successfully.
-verbatim-

you can reproduce this job with 16.6.1.y-val rel_3 and 
+verbatim+
reco_trf.py ami=q121 preexec='rec.dotrigger=false'
-verbatim-

https://svnweb.cern.ch/trac/atlasoff/browser/reconstruction/muonidentification/muoncombinedvalidation/muoncombinedtrackmon/tags/muoncombinedtrackmon-01-00-23/share/muoncombinedtrackvalidation_joboptions.py#l108
+verbatim+
108  if ( muoncombineddqaflags.domuonefficiency ):
109	     from trackisolationtools.trackisolationtoolsconf import trackisolationtool
110	     trackisotool = trackisolationtool(&quot;trackisolationtool&quot;)
111	     toolsvc += trackisotool
112	     from trigdecisiontool.trigdecisiontoolconf import trig__trigdecisiontool
113	     trigdecisiontool = trig__trigdecisiontool()
114	     toolsvc += trigdecisiontool
115	     mueffimonman = athenamonmanager(name=&quot;muoneffimonmanager&quot;,
116	                                   filekey             = dqmonflags.monmanfilekey(),
117	                                   environment         = dqmonflags.monmanenvironment())
118	     from muoncombinedtrackmon.muoncombinedtrackmonconf import muoneffimontool
119	     muoneffimon = muoneffimontool(name = &quot;muoneffimon&quot;,
120	                                   muoncollection = [&quot;stacomuoncollection&quot;,&quot;muidmuoncollection&quot;],
121	                                   calomuoncollection = &quot;calomuoncollection&quot;,
122	                                   trackcollection = &quot;trackparticlecandidate&quot;)
123	     muoneffimon.trackisolationtool = trackisotool
124	     toolsvc += muoneffimon
125	     mueffimonman.athenamontools += [ muoneffimon ]
126	     topsequence += mueffimonman
-verbatim-

please additionally require 
+verbatim+
dqmonflags.usetrigger() and hasattr(toolsvc, dqmonflags.nametrigdectool())
-verbatim-
i mean, please change l108 as
+verbatim+
if ( muoncombineddqaflags.domuonefficiency and dqmonflags.usetrigger() and hasattr(toolsvc, dqmonflags.nametrigdectool()) ):
-verbatim-

this incorrect trigdecisiontool configuration has been seen in 
some muon monitors and muoncombinedtrackmon does once more.........



thanks in advance, 
susumu