Message194460
| Author |
serhiy.storchaka |
| Recipients |
ezio.melotti, mark.dickinson, packetslave, serhiy.storchaka, terry.reedy |
| Date |
2013年08月05日.08:20:11 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1375690813.22.0.207825558113.issue14323@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
> Can you add a reference for the coefficients?
I have only link to Wikipedia which refers to Code of Federal Regulations §73.682. This link (http://en.wikipedia.org/wiki/YIQ) already mentioned at the top of the file.
> (You claim about the current rounding is not exactly correct. While .28*g rounds .277 rather than .274, the current .52*g rounds the non-FCC .523 rather than the FCC .5251. So I avoided making the claim in the suggested entry. It is not important.)
A sum of coefficients in this line should be 0 (Q=0 for R=G=B).
Patch updated. I added a What's New entry and update to use of unittest.main(), rewrite rgb_to_yiq() in the form as in Wikipedia (it uses less multiplications) and write coefficients in yiq_to_rgb() with maximal precision (as calculated with Python). |
|