-
Notifications
You must be signed in to change notification settings - Fork 17
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
RuntimeWarnings #9
Comments
This doesn't indicate anything wrong with the code. There are some weird-but-correct edge cases in the underlying conversion formulas that trigger warnings, and I've been too lazy to track down exactly where we should be suppressing the warnings... In the first case, I think what's happening is that there's a computation that goes like The other ones appear to be the normal warnings you'd expect numpy to emit whenever it produces a nan, and in this case the answers legitimately are nan, so I guess the warnings are correct? |
BTW, if you're trying to find the edges of the sRGB cube in JCh space, then it's probably simpler to start with the edges in sRGB and then convert to JCh instead of going the other way around :-). That's how the animated volumes in the scipy 2015 talk were produced. |
Yes, that's what I meant.
For a given J and h, I'm trying to iterate C from 0 until I hit the RGB wall, for generating bivariate colormaps with maximum chroma that can fit on the screen (and for a given illuminant or whatever) BIDS/colormap#6 (comment) Previously I had it working with either |
(Got it working by turning NaNs into zeros: BIDS/colormap#6 (comment)) (Also shouldn't |
I don't think this was a goal of the CIECAM02 designers, no. In color theory the concept of "pure white" is extremely complicated: https://en.wikipedia.org/wiki/White_point |
there's a problem: if you convert all 2^24 RGB points to JCh, then groupby J, h -> Cmax in 101 x 360 bins Why: if you inflate a tilted rgb cube, black - white vertical, into a cylinder, |
I'm converting lots of colors to find the edges of the RGB cube, and getting
RuntimeWarning
sometimes, which probably shouldn't happen?etc.
The text was updated successfully, but these errors were encountered: