Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error message for SiPixelPhase1TrackClusters DQM #23475

Closed
fabiocos opened this issue Jun 5, 2018 · 11 comments
Closed

Error message for SiPixelPhase1TrackClusters DQM #23475

fabiocos opened this issue Jun 5, 2018 · 11 comments

Comments

@fabiocos
Copy link
Contributor

fabiocos commented Jun 5, 2018

I notice that the test log files in 10_2_X are filled in the (RECO,DQM) step with error messages

%MSG-e SiPixelPhase1TrackClusters: SiPixelPhase1TrackClusters:hltSiPixelPhase1TrackClustersAnalyzer 31-May-2018 11:01:00 CEST Run: 315489 Event: 47976080
incompatible configuration 27!=28
%MSG

check for instance wf 136.85 . This message comes from

https://cmssdt.cern.ch/lxr/source/DQM/SiPixelPhase1Track/plugins/SiPixelPhase1TrackClusters.cc#0105

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 5, 2018

A new Issue was created by @fabiocos Fabio Cossutti.

@davidlange6, @Dr15Jones, @smuzaffar, @fabiocos can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@fabiocos
Copy link
Contributor Author

fabiocos commented Jun 5, 2018

assign dqm

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 5, 2018

New categories assigned: dqm

@jfernan2,@vazzolini,@vanbesien,@kmaeshima,@dmitrijus you have been requested to review this Pull request/Issue and eventually sign? Thanks

@fabiocos
Copy link
Contributor Author

fabiocos commented Jun 5, 2018

In the move from pre3 to pre4 the message has changed from

%MSG-w SiPixelPhase1TrackClusters: SiPixelPhase1TrackClusters:hltSiPixelPhase1TrackClustersAnalyzer 05-Jun-2018 12:05:58 CEST Run: 1 Event: 4
PixelClusterShapeCache collection is not valid
%MSG

to

%MSG-e SiPixelPhase1TrackClusters: SiPixelPhase1TrackClusters:hltSiPixelPhase1TrackClustersAnalyzer 05-Jun-2018 12:05:25 CEST Run: 1 Event: 3
incompatible configuration 27!=28
%MSG

I think that the cause for these messages should be anyway removed at the root, having the log files plenty of them makes difficult to catch new problems, besides it looks clear that there are pieces of software that do not work properly

@fabiocos fabiocos changed the title Error message for SiPixelPhase1TrackClusters Error message for SiPixelPhase1TrackClusters DQM Jun 5, 2018
@boudoul
Copy link
Contributor

boudoul commented Jun 5, 2018

sounds like a deja-vu (even multiple dejavu) sigh ... maybe this thread could help https://groups.cern.ch/group/cms-hlt/Lists/Archive/DispForm.aspx?ID=2999

@fioriNTU
Copy link
Contributor

fioriNTU commented Jun 5, 2018

@fabiocos (@jandrea and @arossi83) this is the usual issue with the HLT monitoring of PixelPhase1. My guess is that e new plot has been added to the Phase1TrackCluster plugin but has not been added in the corresponding HLT DQM configuration (and disabled there). As far as I know TkDQM is already working to provide a completely independent HLT plugin in a way to avoid this kind of things forever

@fabiocos
Copy link
Contributor Author

fabiocos commented Jun 6, 2018

@fioriNTU @boudoul @jandrea @arossi83 thanks for the comments, do you know when is this plugin expected to arrive? It will be in time for CMSSW_10_2_0_pre6?

@arossi83
Copy link
Contributor

arossi83 commented Jun 6, 2018

@fabiocos most probably it will be not ready for CMSSW_10_2_0_pre6, we will provide a FIX for this in time for CMSSW_10_2_0_pre6 @jandrea @fioriNTU

@fabiocos
Copy link
Contributor Author

The PR #23575 "fixes" the LogError problem, but it does not solve ultimately the problem as the module is still not executed (producing a new LogWarning message). Therefore I keep this issue open to track the continuation of the work needed to finally clean up the use of this DQM code.

cmsbuild added a commit that referenced this issue Jun 29, 2018
Fix issue #23475 in the hlTPixel plugin [backport to 10_1_X of #23575)
@fioriNTU
Copy link
Contributor

@fabiocos this issue should be solved by PR #23576 , can we close it?

@fabiocos
Copy link
Contributor Author

@fioriNTU that specific problem looks gone, yes, I close the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants