color HSV constructor

[BlenderNPR]

There are hsv member variables in Color type, yet there is no constructor to put hsv into use.
Facing this problem:
color(1,1,1,1) << in rgba
color(1,1,1,1) << in hsva?

Need hsv constructor, please. Thank 😁

HSV is more intuitive compared to RGB.

[groud]

You already have a set_hsv() function, we cannot define two constructors with the same signature.

[vnen]

color(1,1,1,1) << in rgba
color(1,1,1,1) << in hsva?

The problem is how to differentiate between the two? It's impossible to use the same constructor. Though we have Color8 and ColorN constructors, maybe we can have ColorHSV?

[groud]

maybe we can have ColorHSV?

It would make sense IMO.

[akien-mga]

The problem with Color8, ColorN and ColorHSV is that they're hacky GDScript-specific methods. It would be better to have proper static constructors like Color Color.from_hsv(h, s, v) and Color Color(String name, float alpha=1.0). See also #15435 and #14704.

[vnen]

It would be better to have proper static constructors like Color Color.from_hsv(h, s, v) and Color Color(String name, float alpha=1.0).

I agree. The problem is more fundamental because built-in types don't have static functions. I'm not sure if there's a hard limitation for that, but that's what happens currently.

[akien-mga]

Fixed by #15852.