Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reflect that rgb colors are in 0-1 and floats. #52

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

Carreau
Copy link
Collaborator

@Carreau Carreau commented Apr 4, 2024

Use types to ensure this.

Most of the docs was refering to ints, which can lead one to believe it's 0 to 255.

@trallard trallard requested review from gabalafou and trallard and removed request for gabalafou April 8, 2024 16:54
Copy link
Contributor

@gabalafou gabalafou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the clean up!

The assertions to make the type checker happy were throwing me off a little, so I pushed a commit that obviates the need for them. Feel free to revert if you disagree with my changes.

I will leave merging to you.

Use types to ensure this.

Most of the docs was refering to ints, which can lead one to believe
it's 0 to 255.
@Carreau
Copy link
Collaborator Author

Carreau commented Apr 16, 2024

Ok, merging in.

@Carreau Carreau merged commit 284175c into Quansight-Labs:main Apr 16, 2024
5 of 6 checks passed
@Carreau Carreau deleted the fix-types-01 branch April 16, 2024 09:17
@gabalafou
Copy link
Contributor

I think my commit got lost in the squash, but it doesn't look like that was intentional, so I opened a new PR that restores the commit: #55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants