Skip to content
Snippets Groups Projects

Run opencl without pycuda

Merged Stephan Seitz requested to merge seitz/pystencils:run-opencl-without-pycuda into master

Fix #15 (closed)

This includes !76 (merged).

If anyone wants to use textures on OpenCL, we need to decouple TextureInterpolatedField from CUDA.

Edited by Stephan Seitz

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Stephan Seitz changed the description

    changed the description

  • Thanks. Shouldn‘t the other OpenCL tests be skipped if CUDA is not available since they require both OpenCL and CUDA?

    Also, I don‘t fully understand why you‘re testing OpenCL against CUDA. Wouldn‘t it make more sense to test against CPU so the tests work everywhere?

  • Author Maintainer

    Hi. I'll skip the tests if pycuda is not installed.

    I first compared against numpy but assert np.allequal failed with a small error (about 1e-5). I could have increased the epsilon but found that the error is almost zero if the code runs on the same device (CUDA vs OpenCL).

  • Stephan Seitz added 1 commit

    added 1 commit

    • e9605ab2 - Skip OpenCL tests that require pycuda if pycuda is not installed

    Compare with previous version

  • Author Maintainer

    Hi. I'll skip the tests if pycuda is not installed.

    I first compared against numpy but assert np.allequal failed with a small error (about 1e-5). I could have increased the epsilon but found that the error is almost zero if the code runs on the same device (CUDA vs OpenCL).

    So the comparision with CUDA is just a random coincidence when we quickly assembled the OpenCL backend to show a student how pystencils works.

    Edited by Stephan Seitz
  • merged

  • Martin Bauer mentioned in commit 09a59f7f

    mentioned in commit 09a59f7f

Please register or sign in to reply