Skip to content

Add analysis code for several detectors#8

Merged
jshlee merged 53 commits into
CPLUOS:masterfrom
jang00777:Woojin
Mar 13, 2019
Merged

Add analysis code for several detectors#8
jshlee merged 53 commits into
CPLUOS:masterfrom
jang00777:Woojin

Conversation

@jang00777
Copy link
Copy Markdown

  1. ME0 / GEM / CSC / DT / RPC
    • digi / rechit / segment
  2. h_counts and h_area for plotting hit rate comparison plot

Comment thread MuonAnalyser/test/gemsim_on_condor.sh Outdated

# Steps from "runTheMatrix.py -w upgrade -l 10811.0"

cd /cms/scratch/iwatson/GEM/CMSSW_10_3_0_pre3_vanilla/src/GEMSimulation
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why are you using this?

@jshlee
Copy link
Copy Markdown
Collaborator

jshlee commented Feb 22, 2019

why are there slice test stuff in this PR?

try to make better names i.e. instead of HGCalSimTest.cc have MuonDetHitAnalzser.cc and MuonTrackAnalyzer.cc

Comment thread MuonAnalyser/plugins/HGCalSimTest.cc Outdated
initValue();

hEvents_->Fill(1);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i think its better to loop over digis and rechits instead,

@@ -1,545 +0,0 @@
// cd /cms/ldap_home/iawatson/scratch/GEM/CMSSW_10_1_5/src/ && eval `scramv1 runtime -sh` && eval `scramv1 runtime -sh` && scram b -j 10
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

i dont think this should be deleted. doesnt @tt8888tt use this?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think its from one of my codes from a long time ago. Im pretty sure DJ can't compile in my home directory

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yep Ian is right

@jshlee jshlee merged commit 06b660d into CPLUOS:master Mar 13, 2019
jang00777 added a commit to jang00777/MuonPerformance that referenced this pull request Mar 14, 2019
Add analysis code for several detectors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants