Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

include upstream fixes for the bugs in my vecsize patches #638

Merged
merged 23 commits into from
Apr 13, 2023

Conversation

valassi
Copy link
Member

@valassi valassi commented Apr 11, 2023

This is to include upstream fixes for the bugs in my vecsize patches, ie fix #629

Still in WIP as I am rerunning tput/tmad tests, but otherwise all is ready

cc @roiser

valassi added 14 commits April 11, 2023 18:24
…amcnlo/mg5amcnlo#23

Not surprisingly, the build fails because of
coupl.inc:10:46:

   10 |       DOUBLE PRECISION G, ALL_G(VECSIZE_MEMMAX)
      |                                              1
Error: Symbol ‘vecsize_memmax’ at (1) has no IMPLICIT type
… successful builds after reverting many vecsize changes

In coupl.inc, add VECSIZE_MEMMAX_COUPL duplicating VECSIZE_MEMMAX from vector.inc

In driver.f and auto_dsig1.f, add vector.inc to provide VECSIZE_MEMMAX
…at VECSIZE_MEMMAX == VECSIZE_MEMMAX_COUPL
…, driver.f to make them easier for upstream codegen
…_COUPL instead of VECSIZE_MEMMAX for coupl_write.inc
…g the unpatches upstream codegen

(The patches do not change - it is the reference over which they are applied that changed)

./CODEGEN/generateAndCompare.sh gg_tt --mad --nopatch
git diff --no-ext-diff -R gg_tt.mad/Source/dsample.f gg_tt.mad/Source/genps.inc gg_tt.mad/Source/vector.inc gg_tt.mad/SubProcesses/makefile > CODEGEN/MG5aMC_patches/PROD/patch.common
git diff --no-ext-diff -R gg_tt.mad/SubProcesses/P1_gg_ttx/auto_dsig1.f gg_tt.mad/SubProcesses/P1_gg_ttx/driver.f gg_tt.mad/SubProcesses/P1_gg_ttx/matrix1.f > CODEGEN/MG5aMC_patches/PROD/patch.P1
git checkout gg_tt.mad
This addresses Olivier's comments in vecsize2 mg5amcnlo/mg5amcnlo#24
(This was originally in branch vecsize3 for MR mg5amcnlo/mg5amcnlo#31, which will be closed)
valassi added 9 commits April 13, 2023 09:01
…me and add Events/.keep to repos - no change otherwise
… 7 and modified 1 for gqttq too)

STARTED  AT Wed Apr 12 15:13:06 CEST 2023
./tput/teeThroughputX.sh -mix -hrd -makej -eemumu -ggtt -ggttg -ggttgg -gqttq -ggttggg -makeclean
ENDED(1) AT Wed Apr 12 17:16:54 CEST 2023 [Status=0]
./tput/teeThroughputX.sh -flt -hrd -makej -eemumu -ggtt -ggttgg -inlonly -makeclean
ENDED(2) AT Wed Apr 12 17:42:37 CEST 2023 [Status=0]
./tput/teeThroughputX.sh -makej -eemumu -ggtt -ggttg -gqttq -ggttgg -ggttggg -flt -bridge -makeclean
ENDED(3) AT Wed Apr 12 17:52:16 CEST 2023 [Status=0]
./tput/teeThroughputX.sh -eemumu -ggtt -ggttgg -flt -rmbhst
ENDED(4) AT Wed Apr 12 17:55:29 CEST 2023 [Status=0]
./tput/teeThroughputX.sh -eemumu -ggtt -ggttgg -flt -curhst
ENDED(5) AT Wed Apr 12 17:58:40 CEST 2023 [Status=0]

No errors found in logs
… - all ok except for gqttq as expected (madgraph5#630)

(NB: Fortran MEs for simple processes < ggttggg seem much slower on itscrd80 than on itscrd90?)
(Conversely most CUDA/C++ throughputs seem faster on itscrd80)

STARTED AT Wed Apr 12 18:39:36 CEST 2023
ENDED   AT Wed Apr 12 23:21:06 CEST 2023

Status=0

24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_eemumu_mad/log_eemumu_mad_d_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_eemumu_mad/log_eemumu_mad_f_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_eemumu_mad/log_eemumu_mad_m_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttggg_mad/log_ggttggg_mad_d_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttggg_mad/log_ggttggg_mad_f_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttggg_mad/log_ggttggg_mad_m_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttgg_mad/log_ggttgg_mad_d_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttgg_mad/log_ggttgg_mad_f_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttgg_mad/log_ggttgg_mad_m_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttg_mad/log_ggttg_mad_d_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttg_mad/log_ggttg_mad_f_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggttg_mad/log_ggttg_mad_m_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggtt_mad/log_ggtt_mad_d_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggtt_mad/log_ggtt_mad_f_inl0_hrd0.txt
24 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_ggtt_mad/log_ggtt_mad_m_inl0_hrd0.txt
0 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_gqttq_mad/log_gqttq_mad_d_inl0_hrd0.txt
0 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_gqttq_mad/log_gqttq_mad_f_inl0_hrd0.txt
0 /data/avalassi/GPU2023/madgraph4gpuX/epochX/cudacpp/tmad/logs_gqttq_mad/log_gqttq_mad_m_inl0_hrd0.txt
@valassi valassi self-assigned this Apr 13, 2023
@valassi valassi marked this pull request as ready for review April 13, 2023 07:20
@valassi
Copy link
Member Author

valassi commented Apr 13, 2023

This is now complete and ready to be merged.

All tests have passed in the CI. Self merging

@valassi valassi merged commit b4ed22b into madgraph5:master Apr 13, 2023
@valassi valassi changed the title WIP: include upstream fixes for the bugs in my vecsize patches include upstream fixes for the bugs in my vecsize patches Apr 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

My "vecsize" patches have broken mg5amcnlo "launch" for the default Fortran madevent
1 participant