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

Adding cards for single production of B>bH for Run3 #3468

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DaLu0
Copy link
Contributor

@DaLu0 DaLu0 commented Jun 21, 2023

Dears,
My name is Dalia Ramírez from Universidad Iberoamericana (México). We, together with my supervisor Cristina Oropeza, are working with the B2G group.
We would like to produce the samples for our process for Run3 using the model VLQ_v4_4FNS_UFO, using as starting point the following process from Run2:
https://github.com/cms-sw/genproductions/tree/master/bin/MadGraph5_aMCatNLO/cards/production/2017/13TeV/bpbj_LO_M2200_100k_narrow_Single

Could you please help us by telling if there is something to be changed or modified.

Kind regards,
Dalia

@acarvalh
Copy link
Contributor

Hi @DaLu0

  • The updates that need to be made are summarized in the commits from this PR

To the reviewers, I suggested an organization to single VLQ cards in this PR to help with incoming cards and also with the bookkeeping of the channels we need to cover

@DaLu0
Copy link
Contributor Author

DaLu0 commented Jun 23, 2023

Dear @acarvalh
Thanks a lot for your reply.
The cards were added to the /cards/production/13p6TeV/ directory, is this correct?
The beam energy was updated to 6800 GeV.
And, as we are not using madspin card, I think that was the only change to be done.

Please let me know if there is any other comment.
Cheers,
Dalia

@acarvalh
Copy link
Contributor

acarvalh commented Jun 26, 2023

Hi @DaLu0

To me, it seems correct, but the review of the settings is the area of the GEN folks ;-)

I have some more minor comments on the naming convention of the folders, also to allow an easier finding of those afterward,

  • could you rename the folders to match the conventions used on that first PR we had for single VLQ? That could be: bpbj_R3_LO_M600_100k --> BprimeBToBH_LH_narrowLO (please check if my suggestion makes sense and adapt accordingly if not, specially the 'LH')

  • What does R3 stand for?

  • You do not need to add the number of events in the folder name

The real best is that, on top of the uniformization of naming, would be that the reviewers accept my MR to rearrange folders and you add the cards to the subfolder dedicated to the single VLQ

@acarvalh
Copy link
Contributor

Hi @DaLu0
This is merged (thank you @menglu21 !). So please on renaming also move the cards to genproductions/bin/MadGraph5_aMCatNLO/cards/production/13p6TeV/single_VLQ

Provided that the settings are agreed by GEN folks we are good to go!

Copy link
Collaborator

@menglu21 menglu21 left a comment

Choose a reason for hiding this comment

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

besides the comments inline, please follow the suggestion to move them to the suitable directory for better organization

Comment on lines 3 to 9
define q = u c d s u~ c~ d~ s~
define p = g u c d s u~ c~ d~ s~
define j = g u c d s u~ c~ d~ s~
define l+ = e+ mu+
define l- = e- mu-
define vl = ve vm vt
define vl~ = ve~ vm~ vt~
Copy link
Collaborator

Choose a reason for hiding this comment

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

those lines can be removed

@DaLu0
Copy link
Contributor Author

DaLu0 commented Aug 10, 2023

Dear all, sorry for my delay.
I have just added the cards to the corresponding directory (singleVLQ) and created the directory "BprimeBToBH_LH_narrowLO" the cards were corrected, can you please tell me if this is correct?

@menglu21
Copy link
Collaborator

Hi @DaLu0 it seems your cards are not in genproductions/bin/MadGraph5_aMCatNLO/cards/production/13p6TeV/single_VLQ.

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.

3 participants