-
-
Notifications
You must be signed in to change notification settings - Fork 884
Fix for Issue #1505 #1506
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
Fix for Issue #1505 #1506
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1506 +/- ##
=======================================
Coverage 83.52% 83.52%
=======================================
Files 741 741
Lines 32672 32672
Branches 3662 3662
=======================================
Hits 27289 27289
Misses 4669 4669
Partials 714 714
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Perfect. Thanks for getting this fixed so quickly!
Prerequisites
Description
This fixes an Issue when creating the palette: Instead of
this.maxColors
, the constantQuantizerConstants.MaxColors
was used (which can be bigger thanmaxColors
).