Skip to content

Global position and comments#14

Open
bouchamaouihichem wants to merge 3 commits into
nhaubrich:masterfrom
bouchamaouihichem:Global_position_and_comments
Open

Global position and comments#14
bouchamaouihichem wants to merge 3 commits into
nhaubrich:masterfrom
bouchamaouihichem:Global_position_and_comments

Conversation

@bouchamaouihichem

Copy link
Copy Markdown

No description provided.

@nhaubrich

Copy link
Copy Markdown
Owner

Thanks for this Hichem. From the git perspective, it's all good. You've added the reconstructed lambda position which is useful, but the variable we want to add to the CNN is the z position of the hit. Are you making progress on this front?

@bouchamaouihichem

bouchamaouihichem commented Jul 31, 2020 via email

Copy link
Copy Markdown
Author

@nhaubrich

Copy link
Copy Markdown
Owner

My point is that "LambdaVtxPosz" is not the z-position of the hit. We want to give the CNN information on the incident angle between the particles and the tracker layer. The best way we could do this using "pre-tracking" info is the hit's z position. We should do even better using tracking if we give the hit z and the reconstructed lambda's source vertex position because the lambdas are produced over a spread in z and the incident angle is determined entirely by where the particle was produced and where it interacts with the layer.

tree1->Branch("LambdaPt",&LambdaPtKpi);
tree1->Branch("MCLambdaDeltaR",&MCLambdaDeltaR);

// Global position vars of rec Lambda added to tree

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This info is actually already being saved. See

tree1->Branch("LambdaVtxPosx",&LambdaVtxPosx);
tree1->Branch("LambdaVtxPosy",&LambdaVtxPosy);
tree1->Branch("LambdaVtxPosz",&LambdaVtxPosz);
tree1->Branch("LambdaVtxerrx",&LambdaVtxerrx);
tree1->Branch("LambdaVtxerry",&LambdaVtxerry);
tree1->Branch("LambdaVtxerrz",&LambdaVtxerrz);
tree1->Branch("LambdaVtxcxy", &LambdaVtxcxy);
tree1->Branch("LambdaVtxcxz", &LambdaVtxcxz);
tree1->Branch("LambdaVtxcyz", &LambdaVtxcyz);

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.

2 participants