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

appveyor: output test failures #822

Merged
merged 1 commit into from
Apr 18, 2023
Merged

appveyor: output test failures #822

merged 1 commit into from
Apr 18, 2023

Conversation

nim65s
Copy link
Contributor

@nim65s nim65s commented Apr 18, 2023

Description

Output tests failures on appveyor, as it is currently failing in #821, without more hints than:

The following tests FAILED:
	 41 - LAPACK-xeigtstd_dec_in (Failed)
	 85 - LAPACK-xeigtstz_zec_in (Failed)

Checklist

  • The documentation has been updated.
  • If the PR solves a specific issue, it is set to be closed on merge.

@weslleyspereira
Copy link
Collaborator

Thanks for looking into that. However, I am not sure your change had any effects on the output of AppVeyor. Could you explain how the output has changed?

@nim65s
Copy link
Contributor Author

nim65s commented Apr 18, 2023

Sorry, my message was misleading, the useful information is before the summary, ie. :

Before:

[100%] Built target xeigtstz
ctest -j2
Test project C:/projects/lapack/build
[…]
 39/102 Test  #35: LAPACK-xlintstd_dtest_in .........   Passed   12.10 sec
        Start  41: LAPACK-xeigtstd_dec_in
 40/102 Test  #41: LAPACK-xeigtstd_dec_in ...........***Failed    0.48 sec
        Start  42: LAPACK-xeigtstd_ded_in
 41/102 Test  #42: LAPACK-xeigtstd_ded_in ...........   Passed    1.88 sec
        Start  43: LAPACK-xeigtstd_dgg_in
[…]
 83/102 Test  #79: LAPACK-xlintstz_ztest_in .........   Passed   18.47 sec
        Start  85: LAPACK-xeigtstz_zec_in
 84/102 Test  #85: LAPACK-xeigtstz_zec_in ...........***Failed    0.08 sec
        Start  86: LAPACK-xeigtstz_zed_in
 85/102 Test  #86: LAPACK-xeigtstz_zed_in ...........   Passed    0.97 sec
        Start  87: LAPACK-xeigtstz_zgg_in
[…]
102/102 Test #102: LAPACK-xlintstzc_zctest_in .......   Passed    3.40 sec
98% tests passed, 2 tests failed out of 102
Total Test time (real) =  85.55 sec
The following tests FAILED:
	 41 - LAPACK-xeigtstd_dec_in (Failed)
	 85 - LAPACK-xeigtstz_zec_in (Failed)

After:

[100%] Built target xeigtstz
ctest -j2
Test project C:/projects/lapack/build
[…]
 39/102 Test  #35: LAPACK-xlintstd_dtest_in .........   Passed   12.54 sec
        Start  41: LAPACK-xeigtstd_dec_in
 40/102 Test  #41: LAPACK-xeigtstd_dec_in ...........***Failed    0.43 sec
Running: C:/projects/lapack/build/bin/xeigtstd.exe
ARGS= OUTPUT_FILE;C:/projects/lapack/build/TESTING/dec.out;ERROR_FILE;C:/projects/lapack/build/TESTING/dec.out.err;INPUT_FILE;C:/projects/lapack/TESTING/dec.in
Test OUTPUT:
Test ERROR:
CMake Error at C:/projects/lapack/TESTING/runtest.cmake:36 (message):
  Test C:/projects/lapack/build/bin/xeigtstd.exe returned Stack overflow
        Start  42: LAPACK-xeigtstd_ded_in
 41/102 Test  #42: LAPACK-xeigtstd_ded_in ...........   Passed    1.84 sec
[…]
 83/102 Test  #79: LAPACK-xlintstz_ztest_in .........   Passed   20.13 sec
        Start  85: LAPACK-xeigtstz_zec_in
 84/102 Test  #85: LAPACK-xeigtstz_zec_in ...........***Failed    0.08 sec
Running: C:/projects/lapack/build/bin/xeigtstz.exe
ARGS= OUTPUT_FILE;C:/projects/lapack/build/TESTING/zec.out;ERROR_FILE;C:/projects/lapack/build/TESTING/zec.out.err;INPUT_FILE;C:/projects/lapack/TESTING/zec.in
Test OUTPUT:
Test ERROR:
CMake Error at C:/projects/lapack/TESTING/runtest.cmake:36 (message):
  Test C:/projects/lapack/build/bin/xeigtstz.exe returned Stack overflow
        Start  86: LAPACK-xeigtstz_zed_in
 85/102 Test  #86: LAPACK-xeigtstz_zed_in ...........   Passed    1.02 sec
[…]
98% tests passed, 2 tests failed out of 102
Total Test time (real) =  87.25 sec
The following tests FAILED:
	 41 - LAPACK-xeigtstd_dec_in (Failed)
	 85 - LAPACK-xeigtstz_zec_in (Failed)

@nim65s
Copy link
Contributor Author

nim65s commented Apr 18, 2023

So now we know that xeigtstd.exe & xeigtstz.exe are returning stack overflows

@weslleyspereira
Copy link
Collaborator

Ah... that is way better! Thanks!

@langou langou merged commit 6d6189d into Reference-LAPACK:master Apr 18, 2023
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.

3 participants