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

Adding the DQM and Validation packages for the Phase 2 OuterTracker #6467

Merged
merged 70 commits into from
Dec 11, 2014

Conversation

idebruyn
Copy link
Contributor

New packages DQM/Phase2OuterTracker and Validation/Phase2OuterTracker are added. They contain plots based on the TrackTrigger validation for TTClusters and TTStubs.

Timing (per event):
DQM/OuterTrackerMonitorCluster 0.028358 s
DQM/OuterTrackerMonitorStub 0.000449 s
Validation/OuterTrackerCluster 0.000977 s
Validation/OuterTrackerStub 0.000122 s
Validation/OuterTrackerMCTruth 0.001192 s

Largest contribution to timing is due to loading of StackedTrackerGeometry.

@deguio could you please take a look?

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 7, 2014

Pull request #6467 was updated. @cmsbuild, @civanch, @nclopezo, @mdhildreth can you please check and sign again.

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 7, 2014

-1
Tested at: db8b716
When I ran the RelVals I found an error in the following worklfows:

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6467/1458/summary.html

@mark-grimes
Copy link

The cmsbuild bot is running non-upgrade workflows, don't worry about the "-1". I'll test manually.

@mark-grimes
Copy link

merge

Tested with 10000, 10061, 10200, 10261, 12200, 12261, 12600, 12661, 12800, 12861, 13000, 13061, 13600, 13661, 14000, 14061, 14400 and 14461.

cmsbuild added a commit that referenced this pull request Dec 11, 2014
Adding the DQM and Validation packages for the Phase 2 OuterTracker
@cmsbuild cmsbuild merged commit bf1edb9 into cms-sw:CMSSW_6_2_X_SLHC Dec 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants