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

tensorflow-io v0.8.0 release #472

Merged
merged 4 commits into from Oct 16, 2019
Merged

tensorflow-io v0.8.0 release #472

merged 4 commits into from Oct 16, 2019

Conversation

yongtang
Copy link
Member

NOTE: This PR should not be merged until v0.8.0 is released.

This PR is from R0.8 branch and need to do the following:

  1. Wait until tensorflow==1.15.0 is released and update the version in setup.py.
  2. Rebased against master if necessary
  3. Make sure all tests pass
  4. Complete all outstanding items listed in tensorflow-io 0.8.0 release #459 (comment)
  5. Release v0.8.0 with tag created from R0.8 branch
  6. Merge this PR into master branch.
  7. Delete R0.8 branch.

This PR fixes #459

/cc @BryanCutler @terrytangyuan @dmitrievanthony

Signed-off-by: Yong Tang yong.tang.github@outlook.com

@yongtang
Copy link
Member Author

Some of the tests failures are caused by 1.15.0 and 2.0.0 discrepancy which is being addressed in PR #495.

@yongtang
Copy link
Member Author

Rebased with #495 merged. Tests should pass now.

@yongtang
Copy link
Member Author

Still have one more test failure. Will fix shortly.

@yongtang
Copy link
Member Author

All tests pass now.

@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@yongtang
Copy link
Member Author

yongtang commented Oct 14, 2019

RELEASE.md has been updated with the following:

# Release 0.8.0

## Major Features
* `tensorflow_io.json`: JSON Dataset support.
* `tensorflow_io.dicom`: Medical image DICOM format support.
* `tensorflow_io.genome`: DNA Sequence FastQ format support added.
* `tensorflow_io.ffmpeg`: FFmpeg now support selection of substreams.
* `tensorflow_io.ffmpeg`: FFmpeg now support subtitle (captioning).
* `tensorflow_io.ffmpeg`: FFmpeg now support decode video from memory.
* `tensorflow_io.image`: BMP encoding (encode_bmp) support.
* `tensorflow_io.kafka`: Kafka Dataset now support Kafka Schema Registry.
* `tensorflow_io.audio`: WAV format now support 24 bit audio streams.
* `tensorflow_io.text`: Regex capture group (`re2_full_match`) support.
* manylinux2010 compliant on Linux.

## Thanks to our Contributors

This release contains contributions from many people:

Aleksey Vlasenko, Amarpreet Singh, Bryan Cutler, Damien Pontifex, Duke Wang,
Jiacheng Xu, Marcelo Lerendegui, Mark Daoust, Ouwen Huang, Suyash Kumar,
Yong Tang, Yuan Tang, android

We are also grateful to all who filed issues or helped resolve them, asked and
answered questions, and were part of inspiring discussions.

Let me know if I missed any items.

-- update: added additional contributor, and added encode_bmp

@yongtang
Copy link
Member Author

We also need to rebuild the Travis CI.

@yongtang yongtang marked this pull request as ready for review October 16, 2019 18:35
@yongtang
Copy link
Member Author

TensorFlow 1.15.0 has been released. As such this PR has been updated (bumped to tensorflow>=1.15.0,<1.16.0) and is ready for review.

/cc @terrytangyuan @BryanCutler @vlasenkoalexey @dmitrievanthony

Copy link
Member

@BryanCutler BryanCutler left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

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

Looks great!

RELEASE.md Outdated

Aleksey Vlasenko, Amarpreet Singh, Bryan Cutler, Damien Pontifex, Duke Wang,
Jiacheng Xu, Marcelo Lerendegui, Mark Daoust, Ouwen Huang, Suyash Kumar,
Yong Tang, Yuan Tang, android
Copy link
Member

Choose a reason for hiding this comment

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

Who's "android"?

Copy link
Member Author

Choose a reason for hiding this comment

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

@terrytangyuan android (henrytansetiawan).

Copy link
Member

Choose a reason for hiding this comment

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

We should probably used henrytansetiawan instead? I understand that this list is generated automatically but we should probably manually edit it if needed.

Copy link
Member Author

Choose a reason for hiding this comment

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

@terrytangyuan Update.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang
Copy link
Member Author

Looks like 1.15.0 revert back the namespace change and IO> is not allowed anymore (was working with 11.5.0rc3) 😢

As a workaround, I will change IO> to Io so that no python code change needed. Once we release 0.8.0 I will revert back from Io to IO>.

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang
Copy link
Member Author

All tests pass now 🎉 , I will merge this PR, then have a commit to trigger Travis CI cache. The Travis CI cache is becoming a bottleneck, thought this is what we could do at the moment until we have the Bazel remote cache in place (as was discussed in our last meeting).

@yongtang yongtang merged commit a3634a0 into master Oct 16, 2019
@yongtang yongtang deleted the R0.8 branch October 16, 2019 21:43
i-ony pushed a commit to i-ony/io that referenced this pull request Feb 8, 2021
* Update RELEASE.md

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Adjust version to 0.8.0 and match 'tensorflow>=1.15.0,<1.16.0'

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Update README.md for version compatibility

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Change `IO>` to `Io`

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
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.

tensorflow-io 0.8.0 release
4 participants