Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

LSTOD LS/MD Changes #412

Open
wants to merge 5 commits into
base: lstod
Choose a base branch
from

Conversation

Hoobidoobidoo
Copy link

Added LS/MD cut info into the LSTNtuple.root output and fixing the plotting scripts to print new variables from ntuple.

@VourMa
Copy link
Contributor

VourMa commented Jul 30, 2024

Before this PR goes much further, I wanted to remind you that the development now continues over to the SegmentLinking/cmssw repo, so these developments have to be ported there. You should also keep an eye for the renaming updates that have happened/are happening.

I also have a curiosity: Which machine are you testing these developments on? If it is on some machine in Florida, which setup file do you source when before compiling?

@sgnoohc FYI

@ariostas
Copy link
Member

ariostas commented Dec 9, 2024

@Hoobidoobidoo are you still working on this? If so, I'll help you rebase it to the CMSSW side, but let's do that sooner rather than later so that we don't have pending things on ancient versions of the code.

@slava77 maybe we should mark this repo as a read-only archive?

@slava77
Copy link
Contributor

slava77 commented Dec 9, 2024

@slava77 maybe we should mark this repo as a read-only archive?

fine with me

@Hoobidoobidoo Hoobidoobidoo marked this pull request as ready for review December 19, 2024 21:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants