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

Added brightness parameter as fix to error in test.py and the module test function #28

Merged
merged 1 commit into from Jun 17, 2019

Conversation

robgithub
Copy link
Contributor

Found python/examples/test.py had fatal error when run
Traceback (most recent call last): File "test.py", line 41, in <module> r, g, b = [int(c * 0.99) for c in mote.get_pixel(channel + 1, pixel)] ValueError: too many values to unpack
This was duplicated in the python/library/mote/init.py module file

Found that get_pixel was returning RGB+Brightness, so fix was to accept a brightness parameter.

Also updated the documentation python/documentation/Function-reference.md to reference this.

@Gadgetoid Gadgetoid merged commit cee9537 into pimoroni:master Jun 17, 2019
@Gadgetoid
Copy link
Member

Good spot! Thank you for raising a PR.

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.

None yet

2 participants