-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Quantisation #5706
Quantisation #5706
Conversation
1003083
to
c659750
Compare
Hello @Borda! Thanks for updating this PR. There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2021-02-11 11:21:09 UTC |
Codecov Report
@@ Coverage Diff @@
## release/1.2-dev #5706 +/- ##
================================================
- Coverage 88% 88% -0%
================================================
Files 182 184 +2
Lines 13082 13191 +109
================================================
+ Hits 11576 11624 +48
- Misses 1506 1567 +61 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work and amazing new feature !
fyi, moving adding the new models to #5861 |
07c59ea
to
498ee2b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mix of comments :]
cb8dc27
to
92f8cfb
Compare
Co-authored-by: Nicki Skafte <skaftenicki@gmail.com>
Co-authored-by: Carlos Mocholí <carlossmocholi@gmail.com>
Co-authored-by: Carlos Mocholí <carlossmocholi@gmail.com>
Co-authored-by: Carlos Mocholí <carlossmocholi@gmail.com>
a9a7f51
to
1379621
Compare
189cd59
to
b76eee5
Compare
What does this PR do?
Resources:
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:
Did you have fun?
Make sure you had fun coding 🙃