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

tput_test in the CI does nothing? #937

Closed
valassi opened this issue Jul 23, 2024 · 2 comments · Fixed by #938
Closed

tput_test in the CI does nothing? #937

valassi opened this issue Jul 23, 2024 · 2 comments · Fixed by #938
Assignees

Comments

@valassi
Copy link
Member

valassi commented Jul 23, 2024

This is related to PR #918 to fix the couplings issue #826

The patch modifies the code of susy_gg_t1t1, and alows the CI tmad_test to succeed fixing #826

However, the tput test which include runTest.exe should normally fail unless the reference log is regenerated. In PR #918 the CI test succeeds instead.

I had a look
https://github.com/madgraph5/madgraph4gpu/actions/runs/9980359084/job/27581744124?pr=918


*******************************************************************************
*** tput_test susy_gg_t1t1.mad (P1_gg_t1t1x)
*******************************************************************************

Testing in /home/runner/work/madgraph4gpu/madgraph4gpu/epochX/cudacpp/susy_gg_t1t1.mad/SubProcesses/P1_gg_t1t1x

(SKIP cuda because nvcc is missing on this node)

(SKIP cpp512y which is not supported - no avx512vl in /proc/cpuinfo)

(SKIP cpp512z which is not supported - no avx512vl in /proc/cpuinfo)

@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[testsuite_oneprocess.sh] tput_test (susy_gg_t1t1.mad) finished with status=0 (OK) at Wed Jul 17 19:47:14 UTC 2024
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@

My impression is that this did strictly nothing? Or I might be wrong? In any case, the test should at least made a bit more verbose.

@valassi
Copy link
Member Author

valassi commented Jul 23, 2024

Investigating in #938. Here is a bug
https://github.com/madgraph5/madgraph4gpu/actions/runs/10061775347/job/27812750482?pr=938


@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[testsuite_oneprocess.sh] tput_test (gg_tt.mad) starting at Tue Jul 23 15:28:43 UTC 2024
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
Current directory is /home/runner/work/madgraph4gpu/madgraph4gpu/epochX/cudacpp/gg_tt.mad
FPTYPE=d

*******************************************************************************
*** tput_test gg_tt.mad (P1_gg_ttx)
*******************************************************************************

Testing in /home/runner/work/madgraph4gpu/madgraph4gpu/epochX/cudacpp/gg_tt.mad/SubProcesses/P1_gg_ttx
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------


(SKIP cuda because nvcc is missing on this node)
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------

ls: cannot access 'build.hip*': No such file or directory
DEBUG: 
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------
ls: cannot access 'build.cppnone*': No such file or directory

DEBUG: 
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------

ls: cannot access 'build.cppsse4*': No such file or directory
DEBUG: 
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------

ls: cannot access 'build.cppavx2*': No such file or directory
DEBUG: 
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------


(SKIP cpp512y which is not supported - no avx512vl in /proc/cpuinfo)
-------------------------------------------------------------------------------
--- tput_test gg_tt.mad (P1_gg_ttx) BACKEND=backend
-------------------------------------------------------------------------------


(SKIP cpp512z which is not supported - no avx512vl in /proc/cpuinfo)

@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[testsuite_oneprocess.sh] tput_test (gg_tt.mad) finished with status=0 (OK) at Tue Jul 23 15:28:43 UTC 2024
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@

Will fix this tomorrow

valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
…adgraph5#927

Revert "[actions] in .github/workflows, TEMPORARELY reduce CI tests to only one gg_tt "new ci" test while debugging madgraph5#937"
This reverts commit 5b71191.
@valassi valassi linked a pull request Jul 24, 2024 that will close this issue
@valassi
Copy link
Member Author

valassi commented Jul 24, 2024

This is fixed by #938 - closing

@valassi valassi closed this as completed Jul 24, 2024
valassi added a commit to valassi/madgraph4gpu that referenced this issue Jul 24, 2024
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 a pull request may close this issue.

1 participant