Skip to content

Return float values from QColor in getByIndex#2648

Merged
j9ac9k merged 2 commits intopyqtgraph:masterfrom
nickdimitroff:2627-color-index
Mar 13, 2023
Merged

Return float values from QColor in getByIndex#2648
j9ac9k merged 2 commits intopyqtgraph:masterfrom
nickdimitroff:2627-color-index

Conversation

@nickdimitroff
Copy link
Copy Markdown
Contributor

@nickdimitroff nickdimitroff commented Mar 12, 2023

Make ColorMap_getByIndex return floats for RGB values to fit expectations.

Fixes #2627

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Mar 12, 2023

Hi @nickdimitroff

Thanks for the PR! I added one comment regarding the imports in the test code. Once you make that change, I'll approve and run on the CI system 👍🏻

@nickdimitroff nickdimitroff changed the title WIP: Return float values from QColor in getByIndex Return float values from QColor in getByIndex Mar 12, 2023
@nickdimitroff nickdimitroff marked this pull request as ready for review March 12, 2023 21:44
@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Mar 13, 2023

Thanks @nickdimitroff this LGTM! Merging!

@j9ac9k j9ac9k merged commit 97a1bdf into pyqtgraph:master Mar 13, 2023
@nickdimitroff nickdimitroff deleted the 2627-color-index branch March 13, 2023 15:41
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.

ColorMap.getByIndex() returns wrong colors

2 participants