-
Notifications
You must be signed in to change notification settings - Fork 59
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
Parquet tfxio #53
Parquet tfxio #53
Conversation
cc @iindyk |
@iindyk any chance we can merge this one soon and release a new version so I can implement the rest of the changes in TFX repo? |
I'm working on submitting this internally, it's under review. The release follows the schedule though and should happen in 1-2 weeks |
Also, I don't think you'll have to wait for the release for TFX changes, we release in a train and TFX release is after tfx_bsl release |
@iindyk thanks for the update. How would I go about using these changes in tfx to make the changes on that repo? The dependency is defined here https://github.com/tensorflow/tfx/blob/b7bf21c2119bf6c199af98cded02aa668e9fd6d1/tfx/dependencies.py#L109 |
actually yeah, if you'd like these to be released asap, the tfx changes will need to be merged in a window when tfx-bsl is released and tfx - not yet. I (with help from @jay90099) will make sure to merge before the tfx is released if the changes are going to be ready |
Reopening #52 since the changes were deleted in 017c17b