Yeah, that looks better. It's a little messy having all these functions, and you'll notice I had to name the value setter as 'set_hsv_value', because I wasn't comfortable with just plain 'set_value'. I'll rework this and submit a new request.

What about the getters? red, blue, etc... are you OK with me leaving them as separate functions?