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

RelMon updates and fixes #582

Merged
merged 2 commits into from Sep 10, 2013

Conversation

anorkus
Copy link
Contributor

@anorkus anorkus commented Aug 21, 2013

Added updates for: stand-alone mode,different sample comparison (to compare differently named RelVal samples), counting missing histograms inside each directory.
Removed Physics and Info directories from black list.
Fixed histogram hashing function to include full path for hashing name.

Antanas Norkus added 2 commits August 16, 2013 14:49
Signed-off-by: Antanas Norkus <anorkus@cern.ch>
   * stand-alone method - supporting to run on local machine
     and take CSS/Js over HTTP;
   * different sample support - can compare two different samples.
Fixed:
   * directory name hashing - now hashing full path of directory;

Signed-off-by: Antanas Norkus <anorkus@cern.ch>
@ktf
Copy link
Contributor

ktf commented Aug 21, 2013

@nclopezo can you build this (just build).

@ghost ghost assigned nclopezo Aug 21, 2013
@nclopezo
Copy link
Contributor

Hi,

I took these changes on top of CMSSW_7_0_X_2013-08-21-0200, it built:

https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc472/200/console

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @anorkus (Antanas Norkus) for CMSSW_7_0_X.

RelMon updates and fixes

It involves the following packages:

Utilities/RelMon

@thspeer, @slava77 can you please review it and eventually sign? Thanks.

# $Author: dpiparo $
# $Date: 2012/07/03 05:38:00 $
# $Revision: 1.2 $
# $Author: anorkus $
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please simply remove these.

@thspeer
Copy link
Contributor

thspeer commented Sep 6, 2013

Working @thspeer

@thspeer
Copy link
Contributor

thspeer commented Sep 9, 2013

+1
Tested 8434e6e in CMSSW_7_0_X_2013-09-05-0200-582

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 9, 2013

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests.

@ktf
Copy link
Contributor

ktf commented Sep 10, 2013

@anorkus I appreciate the descriptive branch name... ;-)

ktf added a commit that referenced this pull request Sep 10, 2013
@ktf ktf merged commit 781994c into cms-sw:CMSSW_7_0_X Sep 10, 2013
nclopezo pushed a commit to nclopezo/cmssw that referenced this pull request Apr 30, 2014
root: fix compiledata.h CXX; add armv7 support
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

5 participants