Skip to content

Sync Fork from Upstream Repo #20

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

Merged
merged 61 commits into from
May 3, 2020
Merged

Sync Fork from Upstream Repo #20

merged 61 commits into from
May 3, 2020

Conversation

sthagen
Copy link
Owner

@sthagen sthagen commented May 3, 2020

No description provided.

SylwiaOliwia2 and others added 30 commits March 20, 2020 19:32
'Simple Slider Control' section: change method to 'update', add title. 
Add 'Sliders in Plotly Express', 'Methods' sections.
Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>
* fixups to grammar and wording

* fixups

* add dependencies

* fixups

* use html escape

* fixup

Co-authored-by: Nicolas Kruchten <nicolas@plot.ly>
* add examples

* typo fixup

* Fix pytest (#2181)

* Update doc/python/configuration-options.md

Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>

* Update doc/python/configuration-options.md

Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>

* Update doc/python/configuration-options.md

Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>

* Update doc/python/configuration-options.md

Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>

* add dependencies

* fix typo

* put dash code in markdown so it doesn't execute

* use html escape

* tweaks

Co-authored-by: Emmanuelle Gouillart <emma@plot.ly>
Co-authored-by: Nicolas Kruchten <nicolas@plot.ly>
adding repo and branch options to updateplotlyjsdev
No code was changed. Just edited the markdown to show the code block correctly.
Fixed code block in "Displaying Figures Using Dash" on Renderers help page
Co-Authored-By: Nicolas Kruchten <nicolas@plot.ly>
* Lazy imports of graph object hierarchy for Python 3.7+

This involved splitting validators/graph object classes back into separate files

* lazy submodule imports for plotly and plotly.io modules

* Don't force import pandas and numpy if we're only checking whether an object belongs to these packages

If pandas isn't loaded, we don't need to check whether a value is a DataFrame, and this way we
don't pay the pandas import time.

* Don't auto-import all trace type classes

* Don't construct validators up front for every object when object is created.

Create them lazily, and cache then for use across graph objects of the same type

* Test fixes

* Add optional validation using go.validate object as callable of context manager

* Delay additional imports

* dynamic to static docstrings for Figure io methods

Saves 200ms of startup time!

* Update packages/python/plotly/_plotly_utils/importers.py

Co-Authored-By: Emmanuelle Gouillart <emma@plot.ly>

* Use module-level __dir__ function to restore IPython tab completion with Python 3.7+

* Don't fail hierarchy test when ipywidgets not installed

* cut commented print statements

* Test fix, FigureWidget is not expected to be importable when ipywidgets is not installed

Co-authored-by: Emmanuelle Gouillart <emma@plot.ly>
* Remove plotly express from delayed submodule import.
* added statsmodels to dependencies for CI

* hover can be a dict for skipping or formatting hover

* Revert "added statsmodels to dependencies for CI"

This reverts commit 7cf2ad0.

* ordereddict for py2

* delimiter

* ordereddict

* tuple possible in hover_data dict

* comment

* py2

* black

* debug

* debug

* doc, more tests

* address review comments

* short-circuit instead of bypass flag

* be more permissive when accepting data in hover dict

* remove print

* update tutorial

* add docstring to helper function

* changelog

Co-authored-by: Nicolas Kruchten <nicolas@plot.ly>
jonmmease and others added 29 commits April 21, 2020 19:33
In the 'gapminder' example, the current code snippets appear to assign a list to `fig_dict["layout"]["sliders"]` twice. Once when the figure layout is initially created, and then again when more information is added to `sliders_dict`. This dictionary entry is overwritten before the figure is constructed. My proposed change just removes the initial assignment which seems to be of no consequence. Striving for clarity!
typo on subheader
* fix bug in trendline in the case of missing values

* paint it black

* added statsmodels to dependencies for CI

* version for py2

* Update packages/python/plotly/plotly/express/_core.py

Co-Authored-By: Nicolas Kruchten <nicolas@plot.ly>

* extended test to lowess, and more precise check of attribute length

* Update packages/python/plotly/plotly/tests/test_core/test_px/test_trendline.py

Co-authored-by: Nicolas Kruchten <nicolas@plot.ly>
Replace global validate state with Figure level validate flag
* improve sphinx-gallery case to handle the cases of multiple galleries, and separate source directory

* simplified sphinx-gallery scraper

* blacken

* fixed test

* removed unused import
* Mention extra tag in hovertemplate tutorial

Following various interactions on the forum, including https://community.plotly.com/t/tooltip-in-dash-shows-trace-0-how-to-get-rid-of-that/36844/2

* more text
* Makefile: modify docstrings to use temporarily graph_objs

* changed order of import to respect alphabetical order

* graph_obs / graph_objects in template
* upgrade to plotly.js 1.54

* add new files
@sthagen sthagen merged commit 460869c into sthagen:master May 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.

9 participants