Migrating TensorFlowBackend to Qiboml #1479
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1479 +/- ##
==========================================
- Coverage 99.92% 99.74% -0.18%
==========================================
Files 81 80 -1
Lines 11935 11761 -174
==========================================
- Hits 11926 11731 -195
- Misses 9 30 +21
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
MatteoRobbiati
requested review from
BrunoLiegiBastonLiegi,
renatomello and
alecandido
|
@MatteoRobbiati anything else for this PR, can we merge? |
|
This is ready but docs have to be updated still. I would prefer complete this task and then merge, so that docs will be updated with the code. |
| Note that this should be run during Tensorflow initialization in the beginning | ||
| of the script and before creating the qibo backend. | ||
| For similar wariness when using a machine learning backend (such as TensorFlow or Pytorch) | ||
| please refer to the Qiboml documentation. |
There was a problem hiding this comment.
A link here would be probably helpful, even though I don't think we have any right now in qiboml. This is more a reminder to add it, once we have it.
There was a problem hiding this comment.
I think we can add a simple documentation in Qiboml for now, just with these instructions.
|
Can we fix tests and merge? |
Checklist: