Skip to content
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

Decouple from TensorFlow backend #9

Open
iamohcy opened this issue Nov 3, 2022 · 0 comments
Open

Decouple from TensorFlow backend #9

iamohcy opened this issue Nov 3, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@iamohcy
Copy link
Collaborator

iamohcy commented Nov 3, 2022

Current code depends on multiple changes we have made to the TensorFlow codebase. This means that subsequent updates to TensorFlow require a long merge process to incorporate all the new changes, and ensure that they do not result in any changes that affect our implementation. Additionally, many changes are expected to the XLA codebase with the migration towards OpenXLA.

Goal is to reduce the changes to a patch that makes core functions that we modify virtual, so we can create a “plugin” style subclass that modifies the class as necessary. Specifically, implement a “HloEnvComputation” that subclasses the original “HloComputation” class with the new RewritePlan mechanism, we minimize the code changes necessary since we mostly just need to overload the ReplaceUseWith function.

@iamohcy iamohcy self-assigned this Nov 3, 2022
@iamohcy iamohcy added the enhancement New feature or request label Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant