Skip to content

Conversation

@veblush
Copy link
Collaborator

@veblush veblush commented Jan 31, 2026

BUG=n/a

This was referenced Jan 31, 2026
@veblush veblush changed the title [T2] Refactor Group 9z [T2] Refactor Group 9z: Compression Jan 31, 2026
@veblush veblush mentioned this pull request Jan 31, 2026
@veblush veblush force-pushed the t2-9z branch 2 times, most recently from 67b0cd3 to c1a7361 Compare January 31, 2026 17:27
Refactor t2-11z

Refactor t2-19z

Refactor t2-7z
@veblush veblush force-pushed the t2-9z branch 2 times, most recently from cbb79ad to 57fcbbe Compare January 31, 2026 21:07
@veblush veblush requested a review from ddavis-2015 January 31, 2026 22:29
@veblush veblush marked this pull request as ready for review January 31, 2026 22:29
@veblush veblush requested a review from a team as a code owner January 31, 2026 22:29
Copy link
Member

@ddavis-2015 ddavis-2015 left a comment

Choose a reason for hiding this comment

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

I'm really liking these new ASSERT_ macros 👍

int* output_dims_data, const float* expected_output_data,
TfLiteConvParams* conv_params, TFLMRegistration registration,
float* output_data
void ValidateConvFails(TfLiteTensor* tensors, int tensors_size,
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps this should be ValidateConvFailsDuringPrepare to be very specific about where the failure is expected to occur.


#endif // USE_TFLM_COMPRESSION

void ValidateDepthwiseConvFails(int tensors_size, TfLiteTensor* tensors,
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps ValidateDepthwiseConvFailsDuringPrepare to be more concise as to what failure to expect?

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.

3 participants