Skip to content

Updated CREF to handle bias=null for FC,Conv 8x16 CREF code#3403

Closed
unmeshna017 wants to merge 2 commits intotensorflow:mainfrom
unmeshna017:master_fc_conv
Closed

Updated CREF to handle bias=null for FC,Conv 8x16 CREF code#3403
unmeshna017 wants to merge 2 commits intotensorflow:mainfrom
unmeshna017:master_fc_conv

Conversation

@unmeshna017
Copy link
Contributor

  • Streamlined the usage of GetTensorData() and GetOptionaltensorData()
  • Added bias != nullptr condiition check wherever necessary.

- Streamlined the usage of GetTensorData() and GetOptionaltensorData()
- Added bias != nullptr condiition check wherever necessary.
@unmeshna017 unmeshna017 requested a review from a team as a code owner February 5, 2026 12:52
@ddavis-2015
Copy link
Member

ddavis-2015 commented Feb 5, 2026

@unmeshna017 @veblush I'm thinking these changes should be rejected. It is a valid TfLite model in which these operators can have a missing bias tensor.

@unmeshna017
Copy link
Contributor Author

Adding @vp-cad and @joshih-cad as watchers.

@rameshkunasi
Copy link
Contributor

@ddavis-2015 If I understand correctly, this PR preserves the bias handling logic even when bias=nullptr, allowing higher precision bias datatype(Int64 bias type instead of Int32) to be applied in those cases.

@rameshkunasi
Copy link
Contributor

Reopening this PR!

@rameshkunasi
Copy link
Contributor

@unmeshna017 I am unable to reopen this PR, since the branch was force-pushed/recreated. Can you please create a new PR for same changes?

@unmeshna017
Copy link
Contributor Author

unmeshna017 commented Mar 5, 2026

Hi, created a new Pull request for the changes.
#3492

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.

4 participants