-
Notifications
You must be signed in to change notification settings - Fork 3
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
Temporary fixes to support pytorch backend migration #45
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
=======================================
Coverage 66.59% 66.59%
=======================================
Files 11 11
Lines 488 488
=======================================
Hits 325 325
Misses 163 163
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Are these just the recent modifications added in qiboteam/qibo#1450 ? |
Required for the backend migration. |
src/qiboml/backends/__init__.py
Outdated
AVAILABLE_PLATFORMS = [ | ||
"tensorflow", | ||
"pytorch", | ||
] # temporary: to remove once pytorch and tensorflow are migrated and jax is fully working |
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.
what's the plan here? now that we are migrating pytorch, we could get rid of this, right?
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.
I would keep it until Jax is ready, if we still plan to support Jax soon.
This is needed to complete the Pytorch backend migration from Qibo to Qiboml.