Skip to content

Palette variation experiment#124

Merged
JeffGainsNGames merged 5 commits intomainfrom
palette-variation
Jun 8, 2025
Merged

Palette variation experiment#124
JeffGainsNGames merged 5 commits intomainfrom
palette-variation

Conversation

@biosp4rk
Copy link
Copy Markdown
Collaborator

@biosp4rk biosp4rk commented Feb 15, 2025

This is an experiment to produce more color variation when randomizing colors. Instead of rotating the hue of every color in a palette by the same amount, each color can be additionally rotated by a different amount, based on a randomly generated sine wave pattern

I recommend testing with full hue rotation ranges (0-360) to see if there are any palettes that look too weird

@biosp4rk biosp4rk marked this pull request as ready for review March 15, 2025 02:02
@Miepee
Copy link
Copy Markdown
Contributor

Miepee commented Mar 15, 2025

Before I forget writing any feedback here: From the small sample size I tested with this, the result seemed to be the same to slightly worse.

@biosp4rk
Copy link
Copy Markdown
Collaborator Author

Before I forget writing any feedback here: From the small sample size I tested with this, the result seemed to be the same to slightly worse.

When did you test it? I changed how it works in the last commit

I might make a doc with images that compares the normal hue rotation with the extra variation

@Miepee
Copy link
Copy Markdown
Contributor

Miepee commented Mar 16, 2025

When did you test it?

Close to when the PR was initially made. So this last commit was not included.

@JeffGainsNGames
Copy link
Copy Markdown
Contributor

Results from my testing show a good change, Oklab still looks the most "put together" while HSV allowed some wonky suit palettes

Oklab

image
image
image
image

HSV

image
image
image

@JeffGainsNGames JeffGainsNGames merged commit dd40dd3 into main Jun 8, 2025
9 checks passed
@biosp4rk biosp4rk deleted the palette-variation branch December 13, 2025 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants