Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.7k
Update horizontal-bar-charts.md#4982
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
239 commits Select commitHold shift + click to select a range
7eb6dfd
move plotly-geo and chart-studio to their own repos
emilykla6f02b7
Move package metadata into pyproject.toml and all other python code i…
marthacryan7be470b
Cleanup extra changes
marthacryan7170a51
Revert changes to command names
marthacryan6a2d9bc
Add version attr
marthacryan37807da
Add correct output directory for artifacts
marthacryan989e974
remove references to plotly-geo and chart-studio as subdirectories of…
emilykl41cd025
Move version into separate file
marthacryan7fd6870
Attempt to fix version issues
marthacryancdd0204
Attempt to fix version issues
marthacryan85ac208
Merge branch 'master' into move-chart-studio-and-plotly-geo
emilykl3abd6d4
fix: custom category order was hard-coded
MarcoGorelli9043fff
add go.Pie comparison
MarcoGorelli99cbc45
remove unnecessary paragraph
emilykld974535
Merge pull request #5000 from MarcoGorelli/hardcoded-var
emilykl9910078
Merge pull request #4972 from plotly/move-chart-studio-and-plotly-geo
emilykl48cc36a
Merge with master
marthacryan9795406
Add pip install editable to CI config
marthacryan39750d5
Add version attribute
marthacryan937ad46
Remove extra reference to version.py
marthacryan0d06ded
Code format
marthacryanca9aa0a
Explicitly import importlib.metadata
marthacryan32adcc3
move packages/python/plotly to top level
emilyklec7c557
Add __name__ == main wrapper around command line arg parsing
marthacryan6ea167d
update to python 3.9 for dev build to fix importlib error
marthacryan6f40cdc
Remove unnecessary widget version
marthacryancb01379
adjust paths
emilykl2ef2559
Update version of python for dev build CI
marthacryan634c787
Add pip install editable
marthacryan8abdeaf
Add pyproject.toml to conda recipe
marthacryandafa57f
Fix filepath in conda recipe
marthacryanf642b58
update paths in setup.py
emilykl54d076b
update paths in codegen
emilykl76ca2a3
re-add missing geojson data file
emilykl17e0147
add test for loading px datasets
emilyklbfd6d07
add v6 migration to migration guide
emilykl370efcd
Fix accessing of pyproject.toml version in conda recipe
marthacryanb818373
update .gitignore for widget
emilykl7cd04fb
add link to plotly-geo to README
emilykl9e15334
update paths in Makefile
emilykl5813a8a
Merge pull request #5002 from plotly/simplify-dir-structure
emilykldc1b55b
Merge with master
marthacryan011d410
Remove setup.py from root
marthacryan48e7de3
Fix filepath in commands.py
marthacryan6076305
remove
emilyklc85c576
remove tox instructions
emilykld1905cb
Use new variable name for project root in commands.py
marthacryanf68d997
Remove conda from CI and release
marthacryan2a366ef
Add license file and pytest to pyproject.toml
marthacryan69897ae
Fix syntax issues with pyproject.toml
marthacryanbcfa6a7
Update release instructions
marthacryan5d51c80
remove pre-commit file
emilykl86a7e12
Merge pull request #5010 from plotly/more-cleanup
gvwilson1f1dae9
remove versioneer from manifest.in
marthacryan2ca6dae
Pin narwhals version
marthacryana7499c1
Update pyproject.toml
marthacryan56d2096
Remove wheel install from CI
marthacryanac57d7d
Merge branch 'use-pyprojecttoml' of github.com:plotly/plotly.py into …
marthacryanc6992fc
Update pin of pandas instead of narwhals
marthacryan5111cfa
Merge branch 'master' of github.com:plotly/plotly.py into use-pyproje…
marthacryan3a3b7a8
Update horizontal-bar-charts.md
SimaRaha6c2426a
Rob's edits
rl-utility-man0bac79d
Merge branch 'master' into patch-2
rl-utility-man5d539a0
test: working on PyArrow datetime test
gvwilson410d30e
Update plotly/tests/test_optional/test_px/test_px_hover.py
gvwilsonc58e73a
fix: PyArrow datetime issue in test
gvwilson12b47e0
Update plotly/tests/test_optional/test_px/test_px_input.py
gvwilson72bc179
fix: conditionally remove '"template"' key from test JSON
gvwilson15a7028
Move tests out of the plotly directory
marthacryanf3db071
Rename any references to plotly/tests with just tests
marthacryan6acd25a
Move plotly_utils tests into root tests directory
marthacryan0c343ef
Reset filepath to one that will cause import to fail
marthacryana94517c
fix: remove xfail for PyArrow timezone test
gvwilson98f2c8f
Fix filepath in optional_imports test
marthacryan28a5b74
Update filepaths to plotly_utils tests
marthacryan39032fb
Black
marthacryan4573074
Black
marthacryan86bb1a8
Update plotly/tests/test_optional/test_px/test_px_input.py
gvwilson742f592
Update plotly/tests/test_optional/test_px/test_px_hover.py
gvwilson19b611d
Merge pull request #4977 from plotly/updates-to-tests
gvwilson10b2b87
Merge with master
marthacryane41dcab
Merge pull request #4970 from plotly/use-pyprojecttoml
marthacryanf356c8a
Unpin test requirements and use the same requirements for all python …
marthacryan27c0e9d
Remove extra requirements files
marthacryanfab49c7
remove plotly-geo install and orca tests from circleci config
marthacryan73bc82a
Remove _plotly_future_ and all references to it
marthacryane9523e8
Merge pull request #5020 from plotly/remove-plotly-futures
gvwilson397b1ef
pin fiona version for python<=3.8
emilykl93df541
Change md file names to all caps for consistency
marthacryan4a9cc7e
Merge pull request #5021 from plotly/capitalize-md
gvwilson190a6c0
Use correct filepath for README.md in pyproject.toml
marthacryan60ada7c
Merge pull request #5024 from plotly/marthacryan-patch-1
gvwilson24fe0ee
Only install vaex for python<=3.9
marthacryanc2af57b
Prevent running vaex tests if vaex isn't installed
marthacryan7855fa6
Code formatting
marthacryan508135f
Update tests/test_optional/test_px/test_px_input.py
marthacryan7ca7295
try only running vaex with python 3.9
marthacryan71c6a95
Merge branch 'unpin-requirements' of github.com:plotly/plotly.py into…
marthacryand282c49
Set vaex tests to run on only python 3.9
marthacryanebd06f8
Use uv instead of pip for CI download
marthacryand69f1dc
Update dependencies download to use uv
marthacryan84dde9e
Fix plotly-geo install
marthacryan69a39b9
Use correct venv activate step
marthacryan4fc33b5
Run pytest through python to fix dependency issue
marthacryan632cf32
Add uv for doc build
marthacryan5981bfb
Remove flag that doesn't work for uv
marthacryanef4abb8
Only run vaex tests if vaex is installed
marthacryanffb2b25
Remove flag that doesn't work for uv
marthacryanfe8b92e
Remove unnecessary uninstall step
marthacryancfce34f
Make calls to pytest wrapped by python -m
marthacryan82736ad
Merge pull request #5029 from plotly/uv-ci
marthacryanf06ed71
Merge with master
marthacryanfe3740e
Fix handling of vaex not being present
marthacryan323cc80
Merge test and tests folder and clean up files
marthacryana95ed3f
Fix broken filepath
marthacryan3ddf627
Merge pull request #5023 from plotly/update-test-dir
marthacryan719292e
remove extraneous import of ipython display
marthacryancae9d77
remove unnecessary py parameter and add numpy_installed parameter
marthacryancff6314
Delete binder configuration
marthacryan946d96c
Merge branch 'master' into remove-binder
marthacryanbcdc224
Update conditional in circleci config
marthacryanc493669
Fix syntax of conditional
marthacryan1367fa4
Fix conditional
marthacryanf239366
Merge pull request #5031 from plotly/remove-binder
marthacryan7095710
Parameterize jobs
marthacryanacc61b4
Fix python versions
marthacryan8c3bb2e
Fix indentation
marthacryan707bf68
Rename job for readability of CI output
marthacryan777e376
Switch to uv for other tests
marthacryan19e2c9a
Merge branch 'main' of github.com:plotly/plotly.py into unpin-require…
marthacryanf780164
Revert to pip for pandas downgrade
marthacryanf140060
Address review
marthacryanc9215f0
Revert changes to black version
marthacryan4664d15
Add pandas version as parameter
marthacryand9c9c5e
Use pip for downgrade
marthacryan36c0437
Use easier to read name for jobs
marthacryan29f984b
Try different pandas version
marthacryan2a63724
Remove numpy config
marthacryanfcbcc07
update references to master brach
LiamConnors3850056
Merge pull request #5036 from plotly/update-master-references
LiamConnors7b90bb6
Handle pd.NA in clean_to_json_compatible
DavidKatz-il775734e
update changelog
DavidKatz-ilab90b80
Add back in express extra dependencies and required dependencies
marthacryana6ee27c
Merge pull request #5040 from DavidKatz-il/fix-5039
marthacryan75b5106
Remove deleted files from MANIFEST.in
marthacryan954db31
Prevent pandas install step if default
marthacryan037956a
Merge branch 'main' into merge-docs-changes-to-main
LiamConnors156559d
Merge branch 'main' into merge-docs-changes-to-main
LiamConnors45a8fca
Update pyproject.toml
marthacryanf878224
Merge pull request #5044 from plotly/add-express-dep
marthacryan396b066
Merge branch 'main' of github.com:plotly/plotly.py into unpin-require…
marthacryan60d8682
Remove narwhals from dependencies
marthacryanc54a2bd
Merge pull request #5043 from plotly/merge-docs-changes-to-main
LiamConnors5e90e7d
Remove narwhals from core requirements
marthacryand34460e
Add initial test pypi release workflow
marthacryan0f25bd4
Syntax error
marthacryanefe9451
Raise node version and use verbose flag for debugging
marthacryancab519c
Update .circleci/config.yml
marthacryanc29f6a4
Add check for pandas version to confirm correct install
marthacryane54f99c
Use uv for pip
marthacryan2750aa0
Add numpy version pin
marthacryan53dc043
Switch to use uv in core tests as well
marthacryanf5c2bec
Use uv for percy tests
marthacryand3de0b1
Use later version of numpy for scipy compatibility
marthacryanb89f793
Use version of numpy from before switch to uv
marthacryan0a74d1b
Use different version of numpy
marthacryan801dde1
Update name of test for consistency with other CI jobs
marthacryan2799251
Merge pull request #5019 from plotly/unpin-requirements
marthacryan84e0eff
remove [all] extra from ipython, and add comment explaining fiona pin
emilykldf55242
remove ipykernel; it's installed by jupyter anyway and we don't impor…
emilyklf2d08bc
bump python versions in ci and add python 3.13
emilykl40c1004
build-doc won't run on python 3.12, rip
emilyklf96c25f
bump python version from 3.9 to 3.12 for percy tests (will this work?)
emilykl02a23ff
remove scipy pin
emilykle8cb1f0
try 3.11 for percy tests
emilykl533f5f7
use python 3.9 for build-doc
emilykl3e0a46c
bump pandas 1 version for Percy tests
emilykl18f8444
re-pin scipy version
emilykl8876fab
Merge pull request #5073 from plotly/test-python-3.13-in-ci
emilyklbb932b5
Merge branch 'main' into update-test-requirements-optional
emilykle3f8c1a
Merge pull request #5072 from plotly/update-test-requirements-optional
emilyklc555d57
suppress DeprecationWarnings when updating template.data
emilyklcf3e901
edit comment
emilykl9243f6d
add tests for mapbox deprecation warnings
emilykl2a0241d
format
emilykl237648d
Remove 6.0.1 from changelog
LiamConnors816e5d8
Add fix to changelog
LiamConnorse07ef35
Merge pull request #5083 from plotly/LiamConnors-patch-3
LiamConnors5fe330d
Merge pull request #5084 from plotly/update-changelog-mar
LiamConnors0968434
Merge pull request #5080 from plotly/fix-bogus-deprecationwarning
emilykl47cfafa
Add issue links to changelog
LiamConnorsb3aab37
modify codegen instead of modifying autogenerated _template.py file
emilykl22aa9ba
format
emilyklded1322
Merge pull request #5086 from plotly/fix-deprecationwarning-fix
emilykl7d7548b
bump plotly.js to 3.0.1
emilykl385c1a7
update changelog
emilykl36a99ce
update package-lock
emilykl5a68ce3
Merge pull request #5088 from plotly/plotlyjs-3.0.1
emilykl40a34d8
Merge pull request #5085 from plotly/tidy-changelog
LiamConnorsdb73f5d
Update CHANGELOG.md
emilykld4bf06a
Merge pull request #5089 from plotly/update-changelog-depr
emilykl58e33fc
version changes for v6.0.1
emilyklff43e4a
Merge pull request #5090 from plotly/release-6.0.1
emilyklaa96405
switched from hard coded data to a df.read_csv from github
rl-utility-manbe0a29e
separated update_xaxes from update_layout
rl-utility-manbc826a2
Merge branch 'main' into merge-recent-changes-main
LiamConnors834d36c
Merge pull request #5091 from plotly/merge-recent-changes-main
LiamConnorse265bf0
Update Plotly version
LiamConnors909f1fb
Merge pull request #5092 from plotly/update-plotly-version
LiamConnorsc0995a7
Remove unused step
LiamConnors53c4770
Merge pull request #5094 from plotly/LiamConnors-patch-3
LiamConnors91c74af
Fix widget initialization issue
jescaladab964dbf
Format basewidget.py
jescalada49567a0
Update CHANGELOG.md
jescalada82c8805
Add handling for if jupyterlab / notebook isn't defined
marthacryan856331c
Fix syntax error
marthacryanae0fbed
Merge pull request #5104 from plotly/jl-bug
marthacryan1bd4257
Add test for _widget_layout and _widget_data
jescalada799eb10
Merge pull request #5102 from jescalada/ipywidgets-integration-fix
marthacryan729ae8c
Fix issue of plotly.js being embedded twice in notebook
marthacryanf39dbfd
Black
marthacryan28aa4f3
removed the (irrelevant) neutral category
rl-utility-man6b70a42
removed unneeded comment
rl-utility-manbf7874e
build
marthacryanaba1f3d
Only run on push to main or release branches
marthacryan4ffb5da
Merge branch 'main' into release-testpypi-main
marthacryand0c2ec4
Add workflow dispatch for manual triggering github actions
marthacryanbe868b4
Merge branch 'release-testpypi-main' of github.com:plotly/plotly.py i…
marthacryan94790d2
Merge pull request #5052 from plotly/release-testpypi-main
marthacryan681d80e
Merge pull request #5112 from plotly/double-plotlyjs
marthacryanebece08
Update doc/python/horizontal-bar-charts.md
rl-utility-mana67ec54
removing extra sentence
rl-utility-man0fb1f67
remove ff.create_choropleth test
emilykl533f504
Merge pull request #5132 from plotly/remove-create-choropleth-test
emilykl9180733
Update CHANGELOG.md
emilykl062809c
Merge pull request #5133 from plotly/changelog-not-change-log
gvwilsonde7f742
Merge branch 'main' into patch-2
LiamConnorsefc4316
Moved language from #4984 per suggestion of @LiamConnors
rl-utility-manbe3b500
Update horizontal-bar-charts.md
rl-utility-manFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
Add test for _widget_layout and _widget_data
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit1bd4257c967fee1b2e704db10adcad3bd64fd38b
There are no files selected for viewing
21 changes: 21 additions & 0 deletionstests/test_core/test_figure_widget_backend/test_validate_initialization.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
from unittest import TestCase | ||
import plotly.graph_objs as go | ||
import pytest | ||
try: | ||
go.FigureWidget() | ||
figure_widget_available = True | ||
except ImportError: | ||
figure_widget_available = False | ||
class TestInitialization(TestCase): | ||
if figure_widget_available: | ||
def test_widget_layout_present_on_init(self): | ||
fig = go.FigureWidget(data=go.Scatter(x=[1, 2], y=[1, 2])) | ||
assert fig._widget_layout != {} | ||
def test_widget_data_present_on_init(self): | ||
fig = go.FigureWidget(data=go.Bar(x=[1, 2], y=[1, 2])) | ||
assert fig._widget_data != [] |
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.