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

fix formatting in SMEFT generation #633

Closed
valassi opened this issue Apr 6, 2023 · 2 comments · Fixed by #632
Closed

fix formatting in SMEFT generation #633

valassi opened this issue Apr 6, 2023 · 2 comments · Fixed by #632
Assignees
@valassi

Comments

@valassi
Copy link
Member

valassi commented Apr 6, 2023

fix formatting in SMEFT generation

see 3cef7bd : these post-gen fixes break for instance "e+0" exponents... the codegen must be fixed where it happens
@valassi valassi self-assigned this Mar 19, 2024
This was referenced Mar 19, 2024
Closed
Merged
valassi added a commit to valassi/madgraph4gpu that referenced this issue Mar 20, 2024
@valassi
[smeft2] in CODEGEN, disable clang-formatting post-fixes for SMEFT, w…
53236ad 
…hich were giving problems both in SMEFT and gg_ttg.sa madgraph5#633

Now gg_ttg.sa builds ok
valassi added a commit to valassi/madgraph4gpu that referenced this issue Mar 20, 2024
@valassi
[smeft2] regenerate gg_ttg.sa - back to normal, fixed both the proble…
38149db 
…m of missing mdl_G__exp__2 and the exponent literals madgraph5#633
valassi added a commit to valassi/madgraph4gpu that referenced this issue Mar 20, 2024
@valassi
[smeft2] in CODEGEN, fix the clang formatting post-fix for smeft madg…
6fc07cc 
…raph5#633 - now both smeft_gg_tttt and gg_ttg are better
valassi added a commit to valassi/madgraph4gpu that referenced this issue Mar 20, 2024
@valassi
[smeft2] regenerate smeft_gg_tttt.sa - now exponent literals are fixed …
9af3fbf 
…madgraph5#633 madgraph5#614, but the G^3 variable needs fixing again

The G^3 assignment does give an issue in HRDCOD=0

In file included from Parameters_SMEFTsim_topU3l_MwScheme_UFO.cc:15:
Parameters_SMEFTsim_topU3l_MwScheme_UFO.h: In function ‘const mg5amcCpu::Parameters_SMEFTsim_topU3l_MwScheme_UFO_dependentCouplings::DependentCouplings_sv mg5amcCpu::Parameters_SMEFTsim_topU3l_MwScheme_UFO_dependentCouplings::computeDependentCouplings_fromG(const fptype_sv&, const double*)’:
Parameters_SMEFTsim_topU3l_MwScheme_UFO.h:591:66: error: conversion from ‘mg5amcCpu::fptype_sv’ {aka ‘__vector(4) double’} to non-scalar type ‘const mgOnGpu::cxsmpl<double>’ requested
  591 |         constexpr cxsmpl<double> mdl_G__exp__3 = ( ( G ) * ( G ) * ( G ) );
      |                                                  ~~~~~~~~~~~~~~~~^~~~~~~~~
@valassi valassi linked a pull request Mar 20, 2024 that will close this issue
Fix builds of SMEFT gg_tttt #632
Merged
@valassi
Copy link
Member Author

valassi commented Mar 20, 2024

This is fixed by valassi@6fc07cc in PR #632

@valassi valassi closed this as completed Mar 20, 2024
@valassi
Copy link
Member Author

valassi commented Mar 20, 2024

Note: this is relevant to #614 and #616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@valassi valassi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix builds of SMEFT gg_tttt valassi/madgraph4gpu
1 participant
@valassi