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

fix for fastsim #44

Merged
merged 2 commits into from Jul 5, 2013
Merged

fix for fastsim #44

merged 2 commits into from Jul 5, 2013

Conversation

deguio
Copy link
Contributor

@deguio deguio commented Jul 5, 2013

No description provided.

@ktf
Copy link
Contributor

ktf commented Jul 5, 2013

Doh, it does run fastsim... I missed the --fast... Still a more descriptive comment would help.

@deguio
Copy link
Contributor Author

deguio commented Jul 5, 2013

the fix is minimal and needed to adapt to the new syntax, but the double -s is not correct. I was just trying to submit my first pull request with a small change. going to reject it in the topic collector.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 5, 2013

The following categories have been rejected by @deguio: DQM

@cms-git-dqm

@deguio
Copy link
Contributor Author

deguio commented Jul 5, 2013

feedback: I received the email when I rejected the tagset, but not when I submitted it.

@ktf
Copy link
Contributor

ktf commented Jul 5, 2013

Ok. Try to see if you can update this pull request, rather than submitting a new tag. It should reset you signatures at that point.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 5, 2013

The following categories have been signed by @deguio: DQM

@cms-git-dqm

ktf added a commit that referenced this pull request Jul 5, 2013
Fix for FastSim in one of DQM tests.
@ktf ktf merged commit 3b2ab4f into cms-sw:CMSSW_7_0_X Jul 5, 2013
nclopezo pushed a commit to nclopezo/cmssw that referenced this pull request May 5, 2014
modified the way the performance summary page is created
franzoni pushed a commit to franzoni/cmssw that referenced this pull request May 8, 2014
Handle files with multiple runs in the Harvest job splitting algorithm.
mbluj pushed a commit to mbluj/cmssw that referenced this pull request Apr 27, 2017
Modified Cone-based Tau Builder
mariadalfonso pushed a commit to mariadalfonso/cmssw that referenced this pull request May 6, 2017
tomcornelis pushed a commit to tomcornelis/cmssw that referenced this pull request Feb 15, 2018
merging it even it is a bad policy to merge one's own PR (the commit comes from Yanchu)
amartelli pushed a commit to amartelli/cmssw that referenced this pull request Apr 16, 2019
franzoni pushed a commit to franzoni/cmssw that referenced this pull request May 3, 2021
drankincms pushed a commit to drankincms/cmssw that referenced this pull request Mar 9, 2022
Update to latest L1PF_10_6_1p2_X
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

3 participants