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

Stablehlo to TF saved-model #2157

Merged
merged 2 commits into from
Apr 23, 2024
Merged

Stablehlo to TF saved-model #2157

merged 2 commits into from
Apr 23, 2024

Conversation

sdasgup3
Copy link
Member

@sdasgup3 sdasgup3 commented Apr 3, 2024

Provides an API to convert stablehlo to tf saved model. Added a test to convert a simple linear model.

Things to me done:

  • Provide documentation of the API
  • Test should used the API from the preinstalled stablehlo python package
  • Make sure the API is provoded as part of stablehlo python wheel.

@sdasgup3 sdasgup3 marked this pull request as draft April 3, 2024 21:31
README.md Outdated Show resolved Hide resolved
@sdasgup3 sdasgup3 force-pushed the shlo2tf branch 8 times, most recently from 582fc75 to 379c18a Compare April 8, 2024 16:46
@sdasgup3 sdasgup3 marked this pull request as ready for review April 8, 2024 16:57
@sdasgup3 sdasgup3 force-pushed the shlo2tf branch 3 times, most recently from 2e61e14 to 8daa6fa Compare April 8, 2024 17:46
@sdasgup3 sdasgup3 requested a review from GleasonK April 8, 2024 17:50
README.md Outdated Show resolved Hide resolved
Copy link
Member

@GleasonK GleasonK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM a few minor changes.

@sdasgup3 sdasgup3 requested a review from GleasonK April 23, 2024 14:13
@GleasonK GleasonK assigned sdasgup3 and unassigned GleasonK Apr 23, 2024
@sdasgup3 sdasgup3 merged commit ace3f39 into openxla:main Apr 23, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants