Skip to content

Alpha s part2#422

Closed
roiser wants to merge 35 commits intomadgraph5:masterfrom
roiser:alphaS_part2
Closed

Alpha s part2#422
roiser wants to merge 35 commits intomadgraph5:masterfrom
roiser:alphaS_part2

Conversation

@roiser
Copy link
Member

@roiser roiser commented Apr 6, 2022

A functional complete PR for calculating running alphaS on CPUs. Some more features though will be added in a separate PR for creating more memory buffers and access of the cxtype_sv array within the ME calculation functions. Up to the CPPProcess nothing should change code wise (especially for the bridge interface).

…cales but the Gs over the bridge. calculation of G (or alphaS) on the C++/Cuda side only makes sense when we can pass the calculation as a kernel. LHC experiments depend on a (rather complex) calculation of G/alphaS via LHAPDF --> some work for later.
Conflicts:
	epochX/cudacpp/CODEGEN/PLUGIN/CUDACPP_SA_OUTPUT/output.py
	epochX/cudacpp/gg_tt/SubProcesses/Bridge.h
	epochX/cudacpp/gg_tt/SubProcesses/BridgeKernels.cc
	epochX/cudacpp/gg_tt/SubProcesses/MatrixElementKernels.h
	epochX/cudacpp/gg_tt/SubProcesses/P1_Sigma_sm_gg_ttx/CPPProcess.h
@roiser
Copy link
Member Author

roiser commented Apr 6, 2022

Hi @valassi this is the PR we just talked about, it supersedes #420 , which I will close in a moment. It's functionally complete for CPU calculations, though I will add the additional ways we just discussed.

@roiser roiser mentioned this pull request Apr 6, 2022
@valassi
Copy link
Member

valassi commented Apr 9, 2022

Hi @roiser thanks. I have just merged PR #425, which includs your patch for NEON on Mac and replaces it in a way more consistent to the rest - plus adds many other patches for Mac. I have done this because I saw the NEON patch was included in this alphas part2, and I prefereed to do one thing at a time.

This alphas2 is now the next on my list...

@roiser
Copy link
Member Author

roiser commented Apr 12, 2022

superseded by #428

@roiser roiser closed this Apr 12, 2022
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