-
Notifications
You must be signed in to change notification settings - Fork 140
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
feat(backend): Add Icicle as a GPU backend #800
base: main
Are you sure you want to change the base?
Conversation
7074929
to
76d2d6a
Compare
b9bbe74
to
27abd27
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #800 +/- ##
=======================================
Coverage 94.55% 94.55%
=======================================
Files 154 154
Lines 34587 34587
=======================================
Hits 32705 32705
Misses 1882 1882 ☔ View full report in Codecov by Sentry. |
038b5da
to
5b53639
Compare
e9434db
to
add4616
Compare
For some reason it's not compiling locally. failed to select a version for I'll investigate this a bit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Investigate compilation error
|
||
lambdaworks-gpu = { workspace = true, optional = true } | ||
|
||
[dev-dependencies] | ||
rand = { version = "0.8.5", default-features = false } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be optional. It may not work with no std.
dff2bd3
to
cedb7e4
Compare
Add Icicle as a GPU backend
Description
This pr adds wrappers for conversion of lambaworks types -> Icicle types and adds methods to interface with the Icicle GPU backend. Upstreaming this conversion into the Icicle repo would remove much of the boilerplate. Closes #353
Type of change
Checklist