Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

Allow oversaturated brightness (brightness value >1.0)#29

Closed
jonls wants to merge 1 commit intomasterfrom
oversaturated-brightness
Closed

Allow oversaturated brightness (brightness value >1.0)#29
jonls wants to merge 1 commit intomasterfrom
oversaturated-brightness

Conversation

@jonls
Copy link
Copy Markdown
Collaborator

@jonls jonls commented Feb 1, 2014

This is a test to see how the proposed change in #28 works in practice. After testing this I still think it is a bad idea to introduce (as I argued in the issue) but feel free to convince me otherwise.

@maandree
Copy link
Copy Markdown
Contributor

How about leaving MAX_BRIGHTNESS to 1.0, but make {MIN,MAX}_{BRIGHTNESS,GAMMA} configurable at compile time. I.e. using this commit's changes to colorramps.c and wrap the
definitions of {MIN,MAX}_{BRIGHTNESS,GAMMA} armound #ifndef...#endif directives.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants