Floating-point constants in colorsys.py are undocumented

[Andrew5057]

Documentation

colorsys currently exports three constants - ONE_SIXTH, ONE_THIRD, and TWO_THIRD - that are not mentioned in the docs. A GitHub code search for colorsys.ONE_THIRD shows that they're in use. I'm willing to work on this.

[hugovk]

If they're not documented, and not in __all__, then I think they're not part of the public API?

cpython/Lib/colorsys.py

Lines 24 to 31 in ebf6d9c

	__all__ = ["rgb_to_yiq","yiq_to_rgb","rgb_to_hls","hls_to_rgb",
	"rgb_to_hsv","hsv_to_rgb"]
	# Some floating-point constants
	ONE_THIRD = 1.0/3.0
	ONE_SIXTH = 1.0/6.0
	TWO_THIRD = 2.0/3.0

There's not a huge amount of use, but we don't need to deprecate or rename as like _ONE_SIXTH.

And they're trivial constants, I'm not sure we need to document them, people can define their own if needed.

[StanFromIreland]

+1 to everything Hugo said, and additionally they have no relation to color conversion, such an API does not belong in the module. As such, they are (should be private) helpers.

[hugovk]

Thanks anyway, let's close.