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

CTPPS: DQM for TOTEM RP - backport of #14034 #14696

Merged
merged 10 commits into from Jun 20, 2016

Conversation

grzanka
Copy link
Contributor

@grzanka grzanka commented May 31, 2016

Backport of PR #14034 from CMSSW 81X

The code will not compile as it depends on many features not yet integrated in CMSSW. I'm issuing the PR so as people can start looking at the code. Compilable code is available in the CTPPS repository:
https://github.com/CTPPS/ctpps-offline.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @grzanka (Leszek Grzanka) for CMSSW_8_0_X.

It involves the following packages:

DQM/CTPPS

@cmsbuild, @dmitrijus, @vanbesien, @deguio, @davidlange6 can you please review it and eventually sign? Thanks.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@slava77
Copy link
Contributor

slava77 commented May 31, 2016

@cmsbuild please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 31, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/13278/console

@dmitrijus
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

Comparison not run due to Build errors (RelVals and Igprof tests were also skipped)

@cmsbuild
Copy link
Contributor

-1

Tested at: 656c12e

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
bc82047
0bf418a
d4a4144
ba46a23
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14696/13278/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14696/13278/git-merge-result

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

I found follow errors while testing this PR

Failed tests: Build ClangBuild

  • Build:

I found an error when building:

>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/src/MatcherUsingTracksAlgorithm.cc 
>> Compiling  /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/src/MatcherByPullsAlgorithm.cc 
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/plugins/MatcherByPulls.cc 
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/plugins/MuonMCClassifier.cc 
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/plugins/HLTL1MuonMatcher.cc 
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/DQM/CTPPS/plugins/TotemDAQTriggerDQMSource.cc:21:58: fatal error: DataFormats/TotemDigi/interface/TotemFEDInfo.h: No such file or directory
compilation terminated.
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/DQM/CTPPS/plugins/TotemRPDQMSource.cc:31:70: fatal error: Geometry/Records/interface/VeryForwardRealGeometryRecord.h: No such file or directory
compilation terminated.
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/plugins/TriggerObjectFilterByCollection.cc 
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/MuonAnalysis/MuonAssociators/plugins/MatcherUsingTracks.cc 

  • Clang:

I found a compilation error while trying to compile with clang:
I used this command:
scram b vclean && scram build -k -j 16 USER_CXXFLAGS='-fsyntax-only' COMPILER='llvm compile'

1 warning generated.
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/L1Trigger/L1TGlobal/plugins/L1TUtmTriggerMenuESProducer.cc 
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/L1Trigger/L1TGlobal/plugins/BXVectorInputProducer.cc 
1 warning generated.
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/L1Trigger/L1TGlobal/plugins/L1TGlobalSummary.cc 
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/DQM/CTPPS/plugins/TotemDAQTriggerDQMSource.cc:21:10: fatal error: 'DataFormats/TotemDigi/interface/TotemFEDInfo.h' file not found
#include "DataFormats/TotemDigi/interface/TotemFEDInfo.h"
         ^
1 error generated.
>> Compiling edm plugin /build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/L1Trigger/L1TGlobal/plugins/TriggerMenuParser.cc 
/build/cmsbuild/jenkins-workarea/workspace/ib-any-integration/CMSSW_8_0_X_2016-05-29-0000/src/DQM/CTPPS/plugins/TotemRPDQMSource.cc:31:10: fatal error: 'Geometry/Records/interface/VeryForwardRealGeometryRecord.h' file not found


The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
bc82047
0bf418a
d4a4144
ba46a23
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14696/13278/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14696/13278/git-merge-result

@dmitrijus
Copy link
Contributor

please align this with 81x (to make it compile-able)

@grzanka
Copy link
Contributor Author

grzanka commented May 31, 2016

@dmitrijus what do you mean by aligning it with 81x ? it is an exact copy of PR which was merged in 81x.

@dmitrijus
Copy link
Contributor

@grzanka Oh, sorry, you are right - these are the same commits as 81x PR.
However, it needs to compile first before it can be included.

I guess this dependency-PR (#14630) is missing from 80x.

@dmitrijus
Copy link
Contributor

hold

@cmsbuild
Copy link
Contributor

Pull request has been put on hold by @dmitrijus
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

@cmsbuild
Copy link
Contributor

Pull request #14696 was updated. @cmsbuild, @dmitrijus, @vanbesien, @davidlange6 can you please check and sign again.

@grzanka
Copy link
Contributor Author

grzanka commented Jun 17, 2016

@davidlange6 @slava77 could you please cross-check ?

@davidlange6
Copy link
Contributor

please test

On Jun 17, 2016, at 1:04 PM, Leszek Grzanka notifications@github.com wrote:

@davidlange6 @slava77 could you please cross-check ?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 17, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/13574/console

@dmitrijus
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit bef50b5 into cms-sw:CMSSW_8_0_X Jun 20, 2016
@jan-kaspar jan-kaspar deleted the ctpps_dqm_backport branch October 24, 2017 15:49
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.

None yet

6 participants