Skip to content

Conversation

@kattni
Copy link
Contributor

@kattni kattni commented Feb 1, 2020

Allows for providing float object, such as acceleration or gyro data, without needing to convert to string in user code.

@kattni kattni requested a review from a team February 1, 2020 20:57
@jepler
Copy link
Contributor

jepler commented Feb 2, 2020

In chat with kattni, I mentioned that maybe this would affect how bytes objects were displayed. I was wrong about that (as of now, it's an error to pass a bytes object at all), but it does affect how lists and tuples of strings would be displayed.

Before this PR, l.text = ['a', 'b', 'c'] would behave approximately as l.text = 'abc', but after it would show as the bracketed list. Similar for tuples.

I don't think this necessarily means this PR shouldn't be merged, because to me it is very defensible that you could use "anything print()able" with a Label, even if the property is called "text".

@ladyada
Copy link
Member

ladyada commented Feb 2, 2020

yah folks should format() to hearts desire if they dont want str()

@ladyada ladyada merged commit 36e199a into adafruit:master Feb 2, 2020
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Feb 3, 2020
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