-
Notifications
You must be signed in to change notification settings - Fork 29
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
Integrate new autograph package into catalyst #401
Conversation
Substitute tensorflow imports with standalone autograph imports. Inherit from the existing transformer class to reuse upstream code more effectively. This means we now run all AST passes that standard AutoGraph runs. Eliminate the ag_utils module.
1e762af
to
dee257f
Compare
@josh146 From the technical side I'll consider this ready. As soon as the package is released we can move forward with the PR. |
7c6410d seems like an easy change 🤞 |
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.
Just a small change to enable the docs to properly build!
.. which appears to print (or not print) a space character depending on the system.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #401 +/- ##
==========================================
- Coverage 99.56% 99.54% -0.03%
==========================================
Files 52 51 -1
Lines 8510 8485 -25
Branches 568 569 +1
==========================================
- Hits 8473 8446 -27
- Misses 20 21 +1
- Partials 17 18 +1 ☔ View full report in Codecov by Sentry. |
To remove our dependency on tensorflow, we will be releasing a standalone autograph package. This PR makes the necessary changes in Catalyst to use the new package.
Changes include:
diastatic-malt
a hard dependency ofpennylane-catalyst
.[sc-46389]
[sc-57401]