Skip to content

Conversation

ddavis-2015
Copy link
Member

@tensorflow/micro

Update PACK with INT16 support.
Add INT16 unit tests.

This is a copy of PR #2737

bug=fixes #2736

@tensorflow/micro

Update PACK with INT16 support.
Add INT16 unit tests.

This is a copy of PR#2737

bug=fixes tensorflow#2736
Copy link
Contributor

@rameshkunasi rameshkunasi left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Collaborator

@veblush veblush left a comment

Choose a reason for hiding this comment

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

LGTM other than small nitpicks.

@@ -1,4 +1,4 @@
/* Copyright 2022 The TensorFlow Authors. All Rights Reserved.
/* Copyright 2025 The TensorFlow Authors. All Rights Reserved.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please leave the year the file originated.

@@ -1,4 +1,4 @@
/* Copyright 2019 The TensorFlow Authors. All Rights Reserved.
/* Copyright 2025 The TensorFlow Authors. All Rights Reserved.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please leave the year the file originated.

@mergify mergify bot merged commit 7a7a3de into tensorflow:main Aug 20, 2025
142 of 143 checks passed
@ddavis-2015 ddavis-2015 deleted the pr/andresovela/2737 branch August 20, 2025 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add int16 support to PACK/UNPACK
4 participants