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

Kinematic coverage for polarized datasets #2104

Closed
scarlehoff opened this issue Jun 6, 2024 · 1 comment · Fixed by #2109
Closed

Kinematic coverage for polarized datasets #2104

scarlehoff opened this issue Jun 6, 2024 · 1 comment · Fixed by #2109
Assignees
Labels
Polarised Polarised PDF fits

Comments

@scarlehoff
Copy link
Member

scarlehoff commented Jun 6, 2024

While in principle a process type was added, the implementation of the (unpolarized) DIS datasets are slightly different from the polarized ones. Mainly because of the missing kinematic variables (DIS still use k1/k2/k3).

def _dis_xq2map(kin_info):

It might be enough to modify this function to

    x = kin_info.get_one_of("k1", _Vars.x)
    q = kin_info.get_one_of("k2", _Vars.Q)
    return x, q * q

but it requires some testing to make sure that the results are as expected.

@Radonirinaunimi
Copy link
Member

I have already looked into this and fixed it before to generate kinematic plots for a talk (slack-slides). But I couldn't find anymore nor remember what has done back then.

But your suggestion (with a very tiny tweak) works in #2109 and gives the same result as back then (report).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Polarised Polarised PDF fits
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants