Skip to content
This repository has been archived by the owner on Nov 28, 2019. It is now read-only.

Add a show_widgets option #76

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add a show_widgets option #76

wants to merge 3 commits into from

Conversation

digitaltopo
Copy link

  • Add a show_widgets option so users can choose to suppress the output of the paramnb widgets.
  • Add documentation in example notebook

Related to #23, which is stale now.

@digitaltopo digitaltopo changed the title Add a show_widgets option Add a show_widgets option Dec 13, 2017
@jbednar
Copy link
Member

jbednar commented Dec 13, 2017

Thanks! Here show_widgets is a boolean, but for the purposes of #23, should it be something like "display_type", with values "widgets", "table", "text", or None? Here it seems like you'd need None, but for #23 it would be "table" or "text", I think, with some way to display the values even if they aren't widgets.

@digitaltopo
Copy link
Author

@jbednar what would we include inside of the text and table representations? Or are we looking to just output current values?

It may be a bit difficult to output available "options" for each param. For example, a range param, or colorpicker. What would the options be?

If it's just current values then that should be simple. Please confirm, thanks!

@jbednar
Copy link
Member

jbednar commented Dec 20, 2017

Just current values. Widgets serve two purposes in many cases: allowing selection of a value, and declaring what the selected value is. Even if they fail to allow selection, they can at least declare which value is currently associated with the plot shown. I suppose that they actually serve three cases, in that widgets let you see what values are allowed, but I think that third case is less crucial and as you say, difficult to convey without the widget, so I'm ok with failing to cover that third usage.

@digitaltopo
Copy link
Author

@jbednar Added table and text outputs, please take a look.

They still use ipywidgets to output html, but I wanted to avoid bringing in an addition deps like tabulate for the table display.

@philippjfr
Copy link
Member

Oh wow, thanks @digitaltopo. Not sure @jbednar was necessarily suggesting you implement these options but it's definitely very welcome.

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

Successfully merging this pull request may close these issues.

3 participants