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

[Whizard] fix two bugs for W-bosons #15

Merged
merged 1 commit into from
May 2, 2024

Conversation

Zehvogel
Copy link
Contributor

@Zehvogel Zehvogel commented May 2, 2024

BEGINRELEASENOTES
Fixed two bugs for W-bosons

ENDRELEASENOTES

PDG 24 is W+ not W-
+, - are interpreted as operators in Sindarin if not escaped properly. It is easier to use `Wm, Wp`.
See: https://whizard.hepforge.org/manual/manual006.html#sec144
@apricePhy
Copy link
Collaborator

Thanks Leonhard

@apricePhy apricePhy merged commit 3ff7b61 into key4hep:main May 2, 2024
0 of 6 checks passed
@Zehvogel
Copy link
Contributor Author

Zehvogel commented May 2, 2024

That merge was maybe a bit quick? :D It looks like I broke this test:

Checking Run-Cards/Whizard
Process  Difermion_Polarized Files are different for file Difermion_Polarized_ISR_BST.sin
16,17c16,17
< mW = 80.379
< wW = 2.085
---
> mWp = 80.379
> wWp = 2.085
20a21,22
> mW = 80.419 GeV
> wW = 2.0476 GeV


0% tests passed, 1 tests failed out of 1

@apricePhy
Copy link
Collaborator

It "broke" in the right direction. The reference files have to be updated but I will take care of that.

@Zehvogel
Copy link
Contributor Author

Zehvogel commented May 2, 2024

No I think that part should still set mW not mWp it is because I did not touch

replac = ["+", "-", "1", "2", "3"]

It should be enough to add m, and p there (if that does not break something else in turn)

@Zehvogel
Copy link
Contributor Author

Zehvogel commented May 2, 2024

I saw that you added that change but it did indeed manage to break something else, sorry :(

 < mtop = 173
< wtop = 1.4915
---
> mto = 173
> wto = 1.4915

@apricePhy
Copy link
Collaborator

Thanks for catching this. I was not aware that whizard had a different naming convention for the "heavy" particles. Should be resolved now

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