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.

Sign up for GitHub

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

Jump to bottom

Update int16 LSTM test case with zero_point=0 #2171

Merged
merged 4 commits into from
Aug 16, 2023
Merged

Update int16 LSTM test case with zero_point=0 #2171

merged 4 commits into from
Aug 16, 2023

Conversation

advaitjain
Copy link
Member

  • a76d154 has the changes needed to create test data for the current PR. I reverted the changes back to have the script be mostly unchanged (except for some typos and trailing whitespaces).
  • To get the new quantization parameters:
bazel run tensorflow/lite/micro/kernels/testdata:lstm_test_data_generator

And then manually copy over the parameters into lstm_test_data.cc

BUG=http://b/296130372

@advaitjain advaitjain requested a review from a team as a code owner August 15, 2023 23:26
@TFLM-bot TFLM-bot removed the ci:run label Aug 15, 2023
@advaitjain advaitjain mentioned this pull request Aug 15, 2023
Merged
@TFLM-bot TFLM-bot removed the ci:run label Aug 15, 2023
@mergify mergify bot merged commit 3c291cf into tensorflow:main Aug 16, 2023
97 of 99 checks passed
@advaitjain advaitjain deleted the symmetric_int16_test_case_lstm branch August 16, 2023 15:35
advaitjain added a commit to advaitjain/tflite-micro that referenced this pull request Sep 26, 2023
@advaitjain
Restore cell_state scale changed with tensorflow#2171
190b1e2 
See http://b/296130372#comment3 for additional context.

BUG=http://b/296130372
mergify bot pushed a commit that referenced this pull request Sep 26, 2023
@advaitjain
Restore cell_state scale changed with #2171 (#2245)
0fb5805 
See http://b/296130372#comment3 for additional context.

BUG=http://b/296130372
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rascani rascani rascani approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@advaitjain @rascani @TFLM-bot