-
Notifications
You must be signed in to change notification settings - Fork 74k
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
Move autograph into a separate package and into a separate repo #23601
Comments
Indeed, and we plan to do that very soon. Due to time constraints, we've initially moved it inside TF core, and from there we'll extract the reusable parts into a standalone repo. We hope to have that set up by the end of this year. |
Sorry for the late response. The autograph moved to a new repo and it can be accessed as tf.autograph. Thank you. |
Thank you, and everyone involved into it. 🎉 |
It hasn't been moved to a new repo:
|
Yes, this is still an open issue (I think @chunduriv was referring to autograph being available in its own namespace). Mainly because we need someone to set up and maintain a separate repo - otherwise, significant parts of the source code (tensorflow/python/autograph/pyct) are already independent of TF and could be mover rather mechanically. |
Commenting here for visibility. We have made a static copy of the Here is our copy of the More details on how we are using it: |
Hey all, having autograph available as a separate repo would also be super valuable/important from our end. We're in a similar position to @rmshaffer -- we're depending on Autograph in Catalyst, but don't want to have to depend on the full TensorFlow installation for users of our package. Would it be possible to split autograph out -- without the TensorFlow dependency -- into its own PyPI package? We would be very happy to help out here :) |
It's certainly possible to split it out - autograph was designed with that in mind. Besides some dependencies in the high level API that accumulated as technical debt and should be easy to remove, only the operators are TF-specific, so as long as you have replacement for those (which I suspect you want anyway), a split should definitely be doable. I'm not sure if there are any concrete plans to do such a split, so I'd recommend taking the lead and doing it. |
Hello TensorFlow maintainers, I'm currently tackling the separation of TensorFlow and AutoGraph as a contribution to TF. I have a question regarding the preferred code organization, for which I could see the following options:
|
I think the last option makes most sense, at least in the long run, although a clean move would not be easy to achieve. It would probably be most practical to start with a fork, and continue improvements there, while in parallel configuring TF for an external dependency. |
Hi, Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base. The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate. Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space. |
Hi @tilakrayal, thank you for checking in. I don't think the latest TensorFlow release is resolving this issue, which is about separately packaging and releasing one of the sub-components of TensorFlow. |
System information
Describe the problem
I guess autograph should be moved into a separate package and repo. Because I guess it may be possible to add support of other computation graph frameworks there.
The text was updated successfully, but these errors were encountered: