Skip to content

Add a disamgiguating conversion#47

Open
kshyatt wants to merge 2 commits intolkdvos:mainfrom
kshyatt:ksh/conversion
Open

Add a disamgiguating conversion#47
kshyatt wants to merge 2 commits intolkdvos:mainfrom
kshyatt:ksh/conversion

Conversation

@kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Feb 17, 2026

This seems to be needed for conversion of BlockTensorMaps backed by GPU arrays to CuTensorMap -- otherwise an ambiguity error arises.

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.77%. Comparing base (0b3278f) to head (b82f333).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   55.43%   55.77%   +0.33%     
==========================================
  Files          18       18              
  Lines        1407     1420      +13     
==========================================
+ Hits          780      792      +12     
- Misses        627      628       +1     

☔ 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.

@lkdvos
Copy link
Owner

lkdvos commented Feb 17, 2026

Is this not showing up in Aqua because the ambiguity only is there if CUDA is loaded?

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 17, 2026

Yes basically exactly that.

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 17, 2026

lol "amgiguating"

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 20, 2026

Annoying, more ambiguities to fix.

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 20, 2026

OK, I think this is ok to go (but we'll want to fix Project.toml) after QuantumKitHub/TensorKit.jl#375 gets into shape

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 20, 2026

OK, created a more specific TensorMap conversion and moved some common logic into a new function. Also got the storage type thing seemingly resolved...

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 24, 2026

I think everything got resolved here?

@kshyatt
Copy link
Contributor Author

kshyatt commented Feb 27, 2026

Now with a JLArrays test!

@kshyatt kshyatt requested a review from lkdvos February 27, 2026 14:43
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.

2 participants