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.

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

Add INT32 Support to Sub #2720

Open
HemanthSai7 opened this issue Oct 9, 2024 · 3 comments
Open

Add INT32 Support to Sub #2720

HemanthSai7 opened this issue Oct 9, 2024 · 3 comments
Assignees

Comments

@HemanthSai7
Copy link

When trying to deploy the model on ESP32S3, I get a warning called SUB failed to invoke INT32 not supported. I have fixed it and I want to contribute by opening a PR.

@ddavis-2015
Copy link
Member

ddavis-2015 commented Oct 9, 2024

@HemanthSai7

TFLM (aka LRTM) welcomes PR contributions! Please submit your PR at your leisure with these rules in mind:

  • All code must comply with Google C++ style rules
  • All added files (if any) must build through both Makefile and Bazel
  • Unit tests that allow complete code coverage must be submitted

So you would need unit tests in sub_test.cc for INT32 that cover both broadcast and non-broadcast (just like the INT16). Also be aware that if this is your first time submitting code, you will need to sign the CLA. You will be presented with information on how to do this when you submit the PR.

Also, somewhere in the PR description there must be a line bug=#xxxx where xxxx is this issue number.

@HemanthSai7
Copy link
Author

Okay, I will keep these points in mind.

@HemanthSai7
Copy link
Author

HemanthSai7 commented Oct 22, 2024

Hello, for opening the PR I have to follow the steps mentioned in Contributing.md

git remote add HemanthSai7 git@github.com:HemanthSai7/tflite-micro.git
git fetch HemanthSai7

git checkout -b HemanthSai7-tflm-sub-int32 HemanthSai7/<PR branch name>

# make changes and commit to local branch

# push changes to remove branch

git push HemanthSai7 <PR branch name>

# remove the temp remote to clean up your git environment.

git remote rm HemanthSai7

What is the PR branch name?

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

No branches or pull requests

2 participants