-
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
adding David Lange's comments built on rebase of ValidationDoubleCountin... #8850
adding David Lange's comments built on rebase of ValidationDoubleCountin... #8850
Conversation
…tingFix fixing missing files
A new Pull Request was created by @inugent for CMSSW_7_5_X. adding David Lange's comments built on rebase of ValidationDoubleCountin... It involves the following packages: Validation/EventGenerator @vciulli, @covarell, @deguio, @danduggan, @thuer, @cmsbuild, @nclopezo, @bendavid can you please review it and eventually sign? Thanks. |
Note: missing files refers to files missing by git add when setting up this branch (they were in the original pull request 8676). |
in general there is no need to open a new PR. pushing to the same branch updates the existing one. |
DQMHelper dqm(&i); i.setCurrentFolder("Generator/BPhysics"); | ||
Nobj = dqm.book1dHisto("N"+name, "N"+name, 1, 0., 1,"bin","Number of "+name); | ||
particle.Configure(i); | ||
for(unsigned int j=0;j<daughters.size();j++){std::cout << j << std::endl; daughters[j].Configure(i);} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove the cout
+1 |
Comparison is ready The workflows 140.53 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons |
-1 |
replaced by #9765 |
...gFix fixing missing files
Replaces pull 8676