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 bug identity matrix in RGE #104

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Fixed bug identity matrix in RGE #104

merged 1 commit into from
Dec 10, 2024

Conversation

LucaMantani
Copy link
Collaborator

@LucaMantani LucaMantani commented Dec 9, 2024

This RGE fixes a bug that is present in a very niche scenario, i.e.:

  • there is a UV coupling
  • the initial scale and the observable scale are the same

If those conditions are met, an identity matrix was produced containing the UV coupling in the observable scale operators (it was a square matrix). However, this is wrong as the UV coupling does not exist here and there should always be a column of zeros for the UV coupling (we only evolve the WC).
The fix is to simply remove this special case, since wilson already returns the correct thing if you ask to evolve between two scales that are the same.

E.g. if I have a UV coupling "m" which generates Opd and Op, the correct "identity matrix" when init and obs scale are the same should be

      Op  Opd    m
Op   1.0  0.0  0.0
Opd  0.0  1.0  0.0

while in main it would currently produce

      Op  Opd    m
Op   1.0  0.0  0.0
Opd  0.0  1.0  0.0
m    0.0  0.0  1.0

@LucaMantani LucaMantani merged commit a72532f into main Dec 10, 2024
5 checks passed
@LucaMantani LucaMantani deleted the 241209-fix-rge-bug branch December 10, 2024 08:47
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