-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Removed dependencies which were not needed #1974
Conversation
A new Pull Request was created by @venturia for CMSSW_7_0_X. Removed dependencies which were not needed It involves the following packages: CommonTools/TrackerMap @thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please review it and eventually sign? Thanks. |
+1 |
I'm curious if jenkins report will ever have the comparisons done?
|
Hi Slava, I noticed that it failed because there was a glitch in jenkins and the baseline files were not uploaded correctly. I started the tests again so the comparison is triggered again. |
-1 runTheMatrix-results/1000.0_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT/step2_RunMinBias2011A+RunMinBias2011A+TIER0+SKIMD+HARVESTDfst2+ALCASPLIT.log 1001.0 step2 runTheMatrix-results/1001.0_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD/step2_RunMinBias2011A+RunMinBias2011A+TIER0EXP+ALCAEXP+ALCAHARVD.log you can see the results of the tests here: |
Files are missing. @nclopezo can you follow up tests? |
Files are not missing for these two workflows. DAS glitch. The files are at |
Yeah, that's what I meant, actually… In any case, please restart the tests. |
Minor. Bypassing signatures so that we get it into the IB. Complain if not ok. |
Misc cleanups -- Removed dependencies which were not needed
No description provided.