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.

Sign up for GitHub

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

Jump to bottom

[Feature, Proto] Adopt tensordict in gates and circuit #157

Open
dominikandreasseitz opened this issue Apr 10, 2024 · 2 comments
Open

[Feature, Proto] Adopt tensordict in gates and circuit #157

dominikandreasseitz opened this issue Apr 10, 2024 · 2 comments
Assignees
@mlahariya
Labels
feature New feature or request proto

Comments

@dominikandreasseitz
Copy link
Collaborator

https://pytorch.org/tensordict/
@dominikandreasseitz dominikandreasseitz self-assigned this Apr 10, 2024
@awennersteen
Copy link
Member

I think we considered this some time ago, but the fear was that it would complicate non-torch backends.

we didn't really check though

@RolandMacDoland RolandMacDoland added feature New feature or request proto labels Apr 26, 2024
@RolandMacDoland RolandMacDoland changed the title Adopt tensordict in gates and circuit [Feature, Proto] Adopt tensordict in gates and circuit Apr 26, 2024
@mlahariya mlahariya mentioned this issue Dec 4, 2024
Closed
@mlahariya
Copy link
Collaborator

mlahariya commented Dec 4, 2024 •
edited
Loading

Review issue in qadence. pasqal-io/qadence#386

If this gives performance improvement over dictdataloader in ml_tools, use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mlahariya mlahariya

Labels
feature New feature or request proto
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@RolandMacDoland @awennersteen @dominikandreasseitz @mlahariya