Conversation
| im_type = "1" | ||
| elif self.var_info.bits_per_pixel == 16: | ||
| im_type = "RGB" | ||
| elif self.var_info.bits_per_pixel == 32: |
There was a problem hiding this comment.
Hmm... we can probably get away with this since there aren't currently any other XRGB displays on ev3dev devices, but if there ever are, then they won't be usable in color.
There was a problem hiding this comment.
Do you suggest to use platform here instead of bits_per_pixel?
There was a problem hiding this comment.
I suggest using both.
elif self.var_info.bits_per_pixel == 32:
if platform == EV3:
grayscale
else:
really 32-bit
There was a problem hiding this comment.
69accdf does something similar. Tested on the EV3.
| elif self.platform == "ev3" and self.var_info.bits_per_pixel == 32: | ||
| # Post 4.14 kernel | ||
| im_type = "L" | ||
| elif self.platform == "pistorms" and self.var_info.bits_per_pixel == 16: |
There was a problem hiding this comment.
Please only check for the special case of platform on EV3. We use 16bpp on at least 3 different devices.
|
Why is the background yellow? |
|
Probably just a side effect of |
|
I have a hunch that it is because of this line: this fills XRG and leaves B as 0, which would make yellow. |
|
b18428b results in |
|
I saw a comment elsewhere that this or a related fix was released already; what's the status here? |
|
This is waiting for an approval (I think it can be merged). |

Tested on the latest (2018-04-22) stretch image. It works, and is indeed able to produce grayscale images:
Fixes #455