Skip to content

Conversation

pbugnion
Copy link
Member

@pbugnion pbugnion commented Aug 1, 2017

The widget list page currently displays a traceback because the readout_format
for some of the sliders is set to 'i', which is not a valid d3 format (it's
also not a valid Python format type).

Prior to PR #1550, this looked like it was working because d3-format doesn't throw
an exception when the wrong type gets passed in -- it just silently converts it
to the type '', corresponding to 'either decimal or exponent notation, but trim
insignificant zeros'.

The widget list page currently displays a traceback because the readout_format
for some of the sliders is set to 'i', which is not a valid d3 format (it's
also not a valid Python format).

Previously, this didn't result in a traceback because d3-format doesn't throw
an exception when the wrong type gets passed in -- it just silently converts it
to the type '', corresponding to 'either decimal or exponent notation, but trim
insignificant zeros'.
@pbugnion pbugnion requested review from mwcraig and removed request for mwcraig August 1, 2017 07:10
@jasongrout jasongrout merged commit 09a67f8 into jupyter-widgets:master Aug 1, 2017
@jasongrout
Copy link
Member

Thanks!

@pbugnion pbugnion deleted the fix-docs-readout-format branch August 1, 2017 11:25
@jasongrout jasongrout modified the milestone: 7.0 Aug 1, 2017
@github-actions github-actions bot added the resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 12, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants