Thanks for raising the issue.

Looking back at the original commits for this library it looks like it was originally taken from the cpython implementation and then with the next commit changed to 0-255 ints:

• 2b7058c
• bbbcfc3

It's unclear why that change was made, perhaps the original authors intended use needed the 0-255 int colors.

Adding to the confusion the documented return types show float when in actuality the current version returns ints as noted. https://docs.circuitpython.org/projects/colorsys/en/latest/api.html#colorsys.hls_to_rgb

Instead of adding a warning in the implementation notes, I'd be in favor of changing the functionality to match cpython and return floats instead. If application code needs int colors they could convert it outside of this module so that we can keep compatibility to a maximum where possible.

Hello, I didn’t propose a change in the library itself because it would break the existing code using it. ( I can’t resist to insert an xkcd reference here )

However, this is my personnal point of view, and if you are willing to change the behavior itself, please do :)

A classic :)

That is true, and I'll definitely get input from the wider team.

I had a look around for usages in the most prominent places as well. In all of the Learn Guide repo, as well as all Library examples within the bundle there are two usages of colorsys but both appear to be within the context of CPython rather than CircuitPython:

• https://github.com/adafruit/Adafruit_CircuitPython_RGB_Display/blob/bb7c99ac33dcd9c5ea0c81073915330473813c5b/examples/rgb_display_pillow_bonnet_buttons.py#L37 imports PIL a few lines down from here which means CPython.
• https://github.com/adafruit/Adafruit_Learning_System_Guides/tree/main/BLE_Colorific

Those were the only two I found in those places searching on "colorsys" string.