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

Same sign WW dim6 cards #3077

Merged
merged 6 commits into from
Jun 13, 2022
Merged

Same sign WW dim6 cards #3077

merged 6 commits into from
Jun 13, 2022

Conversation

GiacomoBoldrini
Copy link
Contributor

@GiacomoBoldrini
Copy link
Contributor Author

Following the PRs
#2969
#3079

a SMEFTsim model with multiple restriction cards might be needed in https://cms-project-generators.web.cern.ch/cms-project-generators

I tried to add some general restriction cards for the PRs above and also single and pairs of operators from a set of 15. I do not know if adding a restriction card every-time is sustainable but if you need you can find the tarred SMEFTsim model with the restrictions here:

http://gboldrin.web.cern.ch/gboldrin/generators/SMEFTsim_U35_MwScheme_UFO.tar.gz

Thank you! I'm at disposal for anything

GiacomoBoldrini and others added 2 commits January 13, 2022 00:04
from 5 maxjetflavor to 4 maxjetflavor due to mb != 0
@agrohsje
Copy link
Collaborator

Dear @GiacomoBoldrini , I prefer including the restriction cards in the central file so it is easier to keep track. Let's first finish :
#3084 (comment)
and then finish the review of this PR as well as
#2969
#3079 ?
Does this make sense?

@sihyunjeon
Copy link
Collaborator

@GiacomoBoldrini ping

@Saptaparna
Copy link
Contributor

Hi @GiacomoBoldrini : I see this PR: #3084 is connected too. Any news on either front?

@GiacomoBoldrini
Copy link
Contributor Author

Hi sorry for the later reply.

In the last commit i included the restriction card. I also moved back to mb = 0 in the restriction and switched back to 5fs in the run card.

@agrohsje but the issue raised from #3084 are not related to all the PRs you mentioned (as in the PRs we do not change model or anything like that, contrary to the issue). I do not see the connection between the issue and the PRs, maybe i did not understand.

@Saptaparna I have no news regarding #3084 and probably won't work on that in the future as i have no new test to be made and we are converging to the semileptonic-gridpacks through other paths.

@anpicci
Copy link

anpicci commented Apr 22, 2022

Hello everybody,
I'm writing this as contact person of the ssWW VBS analysis with hadronic tau. We are also interested to produce samples with this set of cards, but we would like to update a new reweight_card that is compatible with the needs of aforementioned analysis

VBSssWW_dim6_reweight_card.dat.txt

If for you it is ok to implement it in the gridpack, we would like to add this reweight_card to this PR.

@mpresill also supervisioned the writing of this card.

@Saptaparna
Copy link
Contributor

@anpicci: Since you are adding taus, please commit your process card.

@anpicci
Copy link

anpicci commented May 26, 2022

Hi @Saptaparna , the taus (also the hadronic ones) are already included in the current proc_card, for what both @GiacomoBoldrini and I know about the implementation of taus in madgraph.

This type of implementation of taus was used also for other VBS processes, and in those cases at the end there were also events with hadronic taus. The latter should depend only on the Pythia side, shoudn't it?

@Saptaparna
Copy link
Contributor

Cards look fine to me.
j = g u c d s u~ c~ d~ s~ b b~, 5 = maxjetflavor and the b's are massless. All good, I think!
@menglu21 and @mlizzo please cross check, so we can merge this PR. Thanks!

@mlizzo
Copy link
Collaborator

mlizzo commented Jun 10, 2022

Hi @GiacomoBoldrini , cards look fine to me as well, I have just two minor comments (one is actually for @Saptaparna @GurpreetSinghChahal @menglu21 ) :

  1. I see that your restriction card is not in the central model. It's not a problem for me, but I wonder if it would be better to include it. @agrohsje already raised this comment some time ago, can anyone - who has rights to do it - upload it? I don't think I can honestly.

  2. Since cut_decays is set true, wouldn't it better to generate leptons within |eta| < 3? This should optimize your phase space, although I see that the very same run_card has been used for dim8 operators (https://github.com/cms-sw/genproductions/blob/master/bin/MadGraph5_aMCatNLO/cards/WW_leptonic/WWjj_SS_dim8_ewk_run_card.dat), maybe we want some consistency.

The PR could be merged already for me, but I'll let other people comment on this before proceeding.

@Saptaparna
Copy link
Contributor

Saptaparna commented Jun 10, 2022

Thanks, @mlizzo! I can take care of uploading the model.

@GiacomoBoldrini
Copy link
Contributor Author

Thanks @mlizzo and @Saptaparna to having a look.
I agree with the eta cut that would for sure help the efficiency but as you noticed the run card is identical to the dim-8 one for consistency. We will take that into consideration for future iterations.
If you'd like a 3 > |etal| cut i can add it anyway, let me know.

Best,
Giacomo

@Saptaparna
Copy link
Contributor

@GiacomoBoldrini can you please send me this model: SMEFTsim_U35_MwScheme_UFO-cW_cHWB_cHDD_cHbox_cHW_cHl1_cHl3_cHq1_cHq3_cqq1_cqq11_cqq31_cqq3_cll_cll1_massless or is it regular SMEFT with the restrict card? We need to make that there are no name issues when the gridpack is extracted during central production of samples.

@GiacomoBoldrini
Copy link
Contributor Author

Hello @Saptaparna,

Yes your second answer is correct.
The model is SMEFTsim_U35_MwScheme_UFO and the restriction card I added to this PR (restrict_cW_cHWB_cHDD_cHbox_cHW_cHl1_cHl3_cHq1_cHq3_cqq1_cqq11_cqq31_cqq3_cll_cll1_massless.dat) was copied inside.

@mlizzo
Copy link
Collaborator

mlizzo commented Jun 13, 2022

@GiacomoBoldrini ok, then if no one else has further comments to raise I would ask either @Saptaparna @GurpreetSinghChahal or @menglu21 to merge this PR when they have uploaded your restriction card in the model

@Saptaparna Saptaparna merged commit 00b8b0a into cms-sw:master Jun 13, 2022
@Saptaparna
Copy link
Contributor

Done!

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.

6 participants