Skip to content

appveyor: output test failures#822

Merged
langou merged 1 commit intoReference-LAPACK:masterfrom
nim65s:ci
Apr 18, 2023
Merged

appveyor: output test failures#822
langou merged 1 commit intoReference-LAPACK:masterfrom
nim65s:ci

Conversation

@nim65s
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

weslleyspereira commented Apr 18, 2023

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
Copy Markdown
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
Copy Markdown
Contributor Author

nim65s commented Apr 18, 2023

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

@weslleyspereira
Copy link
Copy Markdown
Collaborator

weslleyspereira commented Apr 18, 2023

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

Reviewers

@weslleyspereira weslleyspereira weslleyspereira approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nim65s @weslleyspereira @langou