adafruit/Adafruit_CircuitPython_MLX90640

Should use simpleio.map_range in mlx90640_pygamer example.

Closed this issue · 3 comments

Talking with @CedarGroveStudios author of https://github.com/CedarGroveStudios/Thermal_Camera, I figured out that for mapping from temperature to colour palette index, I could use simpleio.map_range rather than a custom made function like I did:

This is not blocking, maybe just to put on my todo list, but anybody is welcomed to fix this before me.

The only additional feature that could make sense would be C° to F° conversion of the min and the max, that could be added. Anything else would be advance feature and not good for an example.

Any other code review is welcome.

This issue is fixed with Pull request #7.

OK, wanna close issue?

[EDIT] oh, wait, i see, it's not merged yet. nvm.

Now it's merged :)
Thanks for the update!