Skip to content

[Merged by Bors] - Fixed case of R == G, following original conversion formula - #4383

Closed
fgiordana wants to merge 1 commit into
bevyengine:mainfrom
fgiordana:bugfix/rgba-hsla-conversion
Closed

fgiordana wants to merge 1 commit into
bevyengine:mainfrom
fgiordana:bugfix/rgba-hsla-conversion

Conversation

@fgiordana

Copy link
Copy Markdown
Contributor

https://en.wikipedia.org/wiki/HSL_and_HSV#From_RGB

Objective

Fixes #4382

Solution

  • Describe the solution used to achieve the objective above.
    Fixed conversion formula to account for red and green component being max and equal

Changelog

Fixed RGB -> HSL colorspace conversion

Migration Guide

@github-actions github-actions Bot added the S-Needs-Triage This issue needs to be labelled label Mar 31, 2022
@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen and removed S-Needs-Triage This issue needs to be labelled labels Mar 31, 2022
@mockersf mockersf added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Apr 3, 2022
@cart

cart commented Apr 4, 2022

Copy link
Copy Markdown
Member

bors r+

bors Bot pushed a commit that referenced this pull request Apr 4, 2022
https://en.wikipedia.org/wiki/HSL_and_HSV#From_RGB

# Objective
Fixes #4382

## Solution

- Describe the solution used to achieve the objective above.
Fixed conversion formula to account for red and green component being max and equal
---

## Changelog
Fixed RGB -> HSL colorspace conversion

## Migration Guide


Co-authored-by: Francesco Giordana <fgiordana@netflix.com>
@bors bors Bot changed the title Fixed case of R == G, following original conversion formula [Merged by Bors] - Fixed case of R == G, following original conversion formula Apr 4, 2022
@bors bors Bot closed this Apr 4, 2022
@fgiordana
fgiordana deleted the bugfix/rgba-hsla-conversion branch April 4, 2022 23:01
aevyrie pushed a commit to aevyrie/bevy that referenced this pull request Jun 7, 2022
…ne#4383)

https://en.wikipedia.org/wiki/HSL_and_HSV#From_RGB

# Objective
Fixes bevyengine#4382

## Solution

- Describe the solution used to achieve the objective above.
Fixed conversion formula to account for red and green component being max and equal
---

## Changelog
Fixed RGB -> HSL colorspace conversion

## Migration Guide


Co-authored-by: Francesco Giordana <fgiordana@netflix.com>
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
…ne#4383)

https://en.wikipedia.org/wiki/HSL_and_HSV#From_RGB

# Objective
Fixes bevyengine#4382

## Solution

- Describe the solution used to achieve the objective above.
Fixed conversion formula to account for red and green component being max and equal
---

## Changelog
Fixed RGB -> HSL colorspace conversion

## Migration Guide


Co-authored-by: Francesco Giordana <fgiordana@netflix.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error in colorspace conversion from RGBA to HSLA

5 participants