Skip to content

Conversation

@onkar-sima-ai
Copy link
Contributor

#9187

I have added support for quantized SIN tflite operator and wrote tests for the same.

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

The PR in general LGTM.

I see there are a few linting errors when running this in CI. You can see on this page how to reproduce linting errors locally on your machine: https://tvm.apache.org/docs/contribute/pull_request.html

You can see the errors here as well: https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-9478/2/pipeline

Once we get this PR past CI, I'm happy for this to be merged.

@onkar-sima-ai
Copy link
Contributor Author

onkar-sima-ai commented Nov 9, 2021

Hi @leandron I am not sure if the failure of CI has to do anything with the changes I introduced.
Is this a race condition?
Can you please help me understand why CI failed?

@onkar-sima-ai
Copy link
Contributor Author

Hi @leandron
I ran the failing test in my local, and it passed.
I was not able to reproduce the same error.
So I rebased and pushed again.
Can you please let me know if we should wait to see if CI passes or Is there something wrong with my PR?

@onkar-sima-ai
Copy link
Contributor Author

Hi @leandron The CI has passed, Can you please merge this PR https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-9478/4/pipeline/269/

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @onkar-sima-ai !

@leandron leandron merged commit 7ea78cc into apache:main Nov 14, 2021
mehrdadh pushed a commit to mehrdadh/tvm that referenced this pull request Dec 1, 2021
* added tests for quantized tflite sin operator

* removing unnecessary rsqrt code_block

* resolving linting error
mehrdadh pushed a commit to mehrdadh/tvm that referenced this pull request Dec 1, 2021
* added tests for quantized tflite sin operator

* removing unnecessary rsqrt code_block

* resolving linting error
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
* added tests for quantized tflite sin operator

* removing unnecessary rsqrt code_block

* resolving linting error
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 11, 2022
* added tests for quantized tflite sin operator

* removing unnecessary rsqrt code_block

* resolving linting error
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* added tests for quantized tflite sin operator

* removing unnecessary rsqrt code_block

* resolving linting error
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.

2 participants