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

fixed pdf list for powheg #2792

Merged
merged 1 commit into from
Dec 16, 2020
Merged

Conversation

agrohsje
Copy link
Collaborator

No description provided.

@agrohsje
Copy link
Collaborator Author

Hi @gvonsem . This should fix the problem you pointed out (#2783). Note that I generally re-ordered and assigned 4fs and 5fs properly as well as hessian vs replicas. Do you mind checking?
@covarell can you check my PR? If ok, I will merge.

@agrohsje agrohsje mentioned this pull request Dec 16, 2020
@covarell
Copy link
Contributor

Hello, thanks for taking care of this! Somehow for a long time this list was incorrect, because Jihun's merges always put back the wrong version...

The pdf names and codes look OK to me now, I wonder if the weight numbers could be reordered a little (e.g. in 5f the main pdf has the code 3000, with 2000 coming later, in 4f this is different. But it is not so important.

@agrohsje
Copy link
Collaborator Author

Hi @covarell . Thanks for checking. I didn't want to change the numbers, maybe somebody is using the info, don't know. Do you suggest to re-order the numbers or change?

@covarell
Copy link
Contributor

Can we use the same numbering as MadGraph? That would be ideal. Though I don't think people are really using these weight numbers for analysis, since they changed so many times during Run2.

@agrohsje
Copy link
Collaborator Author

In MG5 the number just goes through:
PDF=23790 MemberID=6

PDF=25410 MemberID=0

@covarell
Copy link
Contributor

OK, mmm, we cannot do that. Then maybe it is not worth changing the order.

@agrohsje agrohsje merged commit 6031d9e into cms-sw:master Dec 16, 2020
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