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.

Sign up for GitHub

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

Jump to bottom

Update pol DIS cards #181

Merged
merged 5 commits into from
Jul 24, 2024
Merged

Update pol DIS cards #181

merged 5 commits into from
Jul 24, 2024

Conversation

giacomomagni
Copy link
Contributor

Here we update the polarized DIS cards which are not up to date in master anymore.

@giacomomagni giacomomagni self-assigned this Jul 3, 2024
@giacomomagni giacomomagni mentioned this pull request Jul 3, 2024
Closed
3 tasks
@giacomomagni
Copy link
Contributor Author

giacomomagni commented Jul 3, 2024 •
edited
Loading

Most of the changes are trivial, only the COMPASS data are now included with the correct kinematics as we have moved form COMPASS (old paper) to COMPASS15 (new paper)

@giacomomagni giacomomagni mentioned this pull request Jul 3, 2024
Merged
Radonirinaunimi
Radonirinaunimi reviewed Jul 5, 2024
View reviewed changes
Copy link
Member

@Radonirinaunimi Radonirinaunimi left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks a lot for this @giacomomagni!!

We should also include in this PR the integrability cards for $\Delta V$ and $\Delta V_3$.

@giacomomagni
Copy link
Contributor Author

We should also include in this PR the integrability cards for ΔV and ΔV3.

I don't have them, could you plese commit them ?

Radonirinaunimi
Radonirinaunimi approved these changes Jul 5, 2024
View reviewed changes
Copy link
Member

@Radonirinaunimi Radonirinaunimi left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

felixhekhorn
felixhekhorn approved these changes Jul 12, 2024
View reviewed changes
Copy link
Contributor

@felixhekhorn felixhekhorn left a comment

Choose a reason for hiding this comment

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

looks fine (although I didn't check each and every one)

@felixhekhorn
Copy link
Contributor

This is correlated with NNPDF/pinefarm#78

@giacomomagni giacomomagni merged commit d076948 into master Jul 24, 2024
@giacomomagni giacomomagni deleted the update_pol_DIS_cards branch July 24, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Radonirinaunimi Radonirinaunimi Radonirinaunimi approved these changes

@felixhekhorn felixhekhorn felixhekhorn approved these changes

Assignees

@giacomomagni giacomomagni

Labels
polarised
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@giacomomagni @felixhekhorn @Radonirinaunimi