Skip to content

Add tests for manipulating GPU data from Python using nanobind with DLPack #1178

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

Open
wants to merge 3 commits into
base: feature/nanobind
Choose a base branch
from

Conversation

jzaia18
Copy link
Contributor

@jzaia18 jzaia18 commented Jun 11, 2025

DO NOT MERGE

This branch contains test code for manipulating GPU tensors from Python using nanobind with DLPack. It is not meant to be directly integrated with lightning, just to illustrate the usage of nanobind for manipulating GPU data.

@jzaia18 jzaia18 requested a review from AmintorDusko June 11, 2025 19:58
Copy link
Contributor

Hello. You may have forgotten to update the changelog!
Please edit .github/CHANGELOG.md with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@jzaia18 jzaia18 self-assigned this Jun 11, 2025
@jzaia18 jzaia18 added the do not merge Do not merge PR until this label is removed label Jun 11, 2025
Copy link

codecov bot commented Jun 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.48%. Comparing base (4ead951) to head (8e3a0cb).

❗ There is a different number of reports uploaded between BASE (4ead951) and HEAD (8e3a0cb). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (4ead951) HEAD (8e3a0cb)
unit_tests 8 4
Additional details and impacted files
@@                  Coverage Diff                  @@
##           feature/nanobind    #1178       +/-   ##
=====================================================
- Coverage             95.71%   27.48%   -68.24%     
=====================================================
  Files                   183       28      -155     
  Lines                 29356     2718    -26638     
=====================================================
- Hits                  28098      747    -27351     
- Misses                 1258     1971      +713     
Flag Coverage Δ
unit_tests 27.48% <ø> (-68.24%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge PR until this label is removed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant