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

PR #179 broke testAPRCuda on 'master' branch #182

Closed
krzysg opened this issue Nov 9, 2023 · 1 comment · Fixed by #183
Closed

PR #179 broke testAPRCuda on 'master' branch #182

krzysg opened this issue Nov 9, 2023 · 1 comment · Fixed by #183
Assignees

Comments

@krzysg
Copy link
Member

krzysg commented Nov 9, 2023

Latest merge of PR #179 broke some cuda tests.

To reproduce:

  • clone latest 'master' of LibAPR
  • run commands:
mkdir build
cd build
cmake -DCMAKE_BUILD_TYPE=Debug -DAPR_USE_CUDA=ON -DAPR_TESTS=ON -DAPR_BUILD_EXAMPLES=OFF ..
make -j 16
ctest

it will fail with:

Test project /tmp/LibAPR/build
      Start  1: fuzz_compress
 1/10 Test  #1: fuzz_compress ....................   Passed    0.01 sec
      Start  2: fuzz_decompress
 2/10 Test  #2: fuzz_decompress ..................   Passed    0.00 sec
      Start  3: testMeshData
 3/10 Test  #3: testMeshData .....................   Passed    0.84 sec
      Start  4: testTiff
 4/10 Test  #4: testTiff .........................   Passed    0.00 sec
      Start  5: testAPR
 5/10 Test  #5: testAPR ..........................   Passed   73.65 sec
      Start  6: testComputeGradient
 6/10 Test  #6: testComputeGradient ..............   Passed    0.85 sec
      Start  7: testLocalIntensityScale
 7/10 Test  #7: testLocalIntensityScale ..........   Passed    0.48 sec
      Start  8: testPullingScheme
 8/10 Test  #8: testPullingScheme ................   Passed    0.08 sec
      Start  9: testAPRParameters
 9/10 Test  #9: testAPRParameters ................   Passed    0.00 sec
      Start 10: testAPRCuda
10/10 Test #10: testAPRCuda ......................***Exception: SegFault  0.16 sec

90% tests passed, 1 tests failed out of 10

Total Test time (real) =  76.07 sec

The following tests FAILED:
	 10 - testAPRCuda (SEGFAULT)
Errors while running CTest
Output from these tests are in: /tmp/LibAPR/build/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.

if testAPRCuda run seperately it will output:

$ ./test/testAPRCuda 
[==========] Running 105 tests from 12 test suites.
[----------] Global test environment set-up.
[----------] 10 tests from CreatDiffDimsSphereTest
[ RUN      ] CreatDiffDimsSphereTest.TEST_GPU_DOWNSAMPLE
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_level.tif]
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_original.tif]
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_pc.tif]
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_x.tif]
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_y.tif]
Opening file: [/tmp/LibAPR/test/files/Apr/sphere_diff_dims/sphere_z.tif]
Segmentation fault (core dumped)

Before latest PR so at:

dca7e7fb 2022-06-16 | Merge pull request #165 from AdaptiveParticles/develop (tag: v2.1.0) [Joel Jonsson]

everything worked fine.

@joeljonsson joeljonsson linked a pull request Nov 9, 2023 that will close this issue
@cheesema
Copy link
Member

cheesema commented Nov 9, 2023

👋🏻 @krzysg

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.

3 participants