-
Notifications
You must be signed in to change notification settings - Fork 28
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
Color Order In GetPalette #6
Comments
Hi, thx for you issue! |
I have tested the Windows Forms Application version and I couldn't find the same problem, my issue was only faced using Xamarin forms iOS, didn't test any other platforms, its only a matter of bytes ordering, I've confirmed the problem by testing 3 images each filled with a major single color, result is red is swapped with blue, temporarily fixed by using .R as the blue byte, . .B as the red byte.
Again.. i gotta say .. great work .. and thanks for your efforts.
…Sent from my iPhone
On Jun 28, 2017, at 9:27 PM, ksemenenko <notifications@github.com<mailto:notifications@github.com>> wrote:
Hi, thx for you issue!
It's only for iOS?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#6 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AOHydJBc-_RZaY5H7CdOsMxI6cDeeZFDks5sIpr6gaJpZM4OIU2e>.
|
Hi @Houri93 please check it on version 1.0.0.4 |
hello again, just updated to the latest version via nuget, the problem still exists as it has been previously |
@Houri93 Thank you. I will then continue to fix this. |
Hello! You could give me a picture and colors that should be the result? |
@KSemenenko `
|
only on Xamarin.forms nuget package:
//var pal = _colorThief.GetPalette(pictureBoxSpectrum.Image, 6, 10, false).ToList();
all the colors are correct, but Red and Blue color Bytes are swapped.
I was porting an app from windows forms app to Xamarin iOS, and noticed that all the colors are not the same, after some investigation turns out only instead of using RGB of your values, used them as BGR.
The text was updated successfully, but these errors were encountered: