Skip to content

Fix tf.io.decode_raw bugs and update documentation.#49095

Merged
mihaimaruseac merged 1 commit intotensorflow:r2.5from
geetachavan1:cherrypicks_Y0H85
May 11, 2021
Merged

Fix tf.io.decode_raw bugs and update documentation.#49095
mihaimaruseac merged 1 commit intotensorflow:r2.5from
geetachavan1:cherrypicks_Y0H85

Conversation

@geetachavan1
Copy link
Contributor

Fixes cases where specifying fixed_length resulted in data loss and even segfault and corruption of the Python interpreter. The fix is subtle but needed due to pointer arithmetic rules.

Makes sure that fixed_length does not change the output when present but not needed.

Eliminates needless copy and cast in the main codepath.

PiperOrigin-RevId: 371322725
Change-Id: I514ef67a2961c86422f69d05122d31615e87896c

Fixes cases where specifying `fixed_length` resulted in data loss and even segfault and corruption of the Python interpreter. The fix is subtle but needed due to pointer arithmetic rules.

Makes sure that `fixed_length` does not change the output when present but not needed.

Eliminates needless copy and cast in the main codepath.

PiperOrigin-RevId: 371322725
Change-Id: I514ef67a2961c86422f69d05122d31615e87896c
@google-cla google-cla bot added the cla: yes label May 10, 2021
@mihaimaruseac mihaimaruseac merged commit ca95f11 into tensorflow:r2.5 May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants