Skip to content
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

Update horizontal-bar-charts.md #4982

Draft
wants to merge 26 commits into
base: doc-prod
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
7eb6dfd
move plotly-geo and chart-studio to their own repos
emilykl Jan 11, 2025
989e974
remove references to plotly-geo and chart-studio as subdirectories of…
emilykl Jan 14, 2025
85ac208
Merge branch 'master' into move-chart-studio-and-plotly-geo
emilykl Jan 29, 2025
3abd6d4
fix: custom category order was hard-coded
MarcoGorelli Jan 30, 2025
9043fff
add go.Pie comparison
MarcoGorelli Jan 30, 2025
99cbc45
remove unnecessary paragraph
emilykl Jan 30, 2025
d974535
Merge pull request #5000 from MarcoGorelli/hardcoded-var
emilykl Jan 30, 2025
9910078
Merge pull request #4972 from plotly/move-chart-studio-and-plotly-geo
emilykl Jan 30, 2025
32adcc3
move packages/python/plotly to top level
emilykl Jan 30, 2025
cb01379
adjust paths
emilykl Jan 30, 2025
f642b58
update paths in setup.py
emilykl Jan 30, 2025
54d076b
update paths in codegen
emilykl Jan 31, 2025
76ca2a3
re-add missing geojson data file
emilykl Feb 3, 2025
17e0147
add test for loading px datasets
emilykl Feb 3, 2025
bfd6d07
add v6 migration to migration guide
emilykl Feb 3, 2025
b818373
update .gitignore for widget
emilykl Feb 4, 2025
7cd04fb
add link to plotly-geo to README
emilykl Feb 4, 2025
9e15334
update paths in Makefile
emilykl Feb 4, 2025
5813a8a
Merge pull request #5002 from plotly/simplify-dir-structure
emilykl Feb 4, 2025
6076305
remove
emilykl Feb 4, 2025
c85c576
remove tox instructions
emilykl Feb 4, 2025
5d51c80
remove pre-commit file
emilykl Feb 4, 2025
86a7e12
Merge pull request #5010 from plotly/more-cleanup
gvwilson Feb 5, 2025
3a3b7a8
Update horizontal-bar-charts.md
SimaRaha Jan 25, 2025
6c2426a
Rob's edits
rl-utility-man Jan 27, 2025
0bac79d
Merge branch 'master' into patch-2
rl-utility-man Feb 6, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
The diff you're trying to view is too large. We only load the first 3000 changed files.
80 changes: 18 additions & 62 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,13 @@ commands:
- run:
name: Install dependencies
command: |
cd packages/python/plotly
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
pip install -r ./test_requirements/requirements_<<parameters.py>>_core.txt
- run:
name: Test core
command: |
cd packages/python/plotly
. venv/bin/activate
pytest plotly/tests/test_core
no_output_timeout: 20m
Expand All @@ -41,7 +39,6 @@ commands:
- run:
name: Install dependencies
command: |
cd packages/python/plotly
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
Expand All @@ -52,47 +49,40 @@ commands:
- run:
name: Install plotly-geo
command: |
cd packages/python/plotly-geo
. ../plotly/venv/bin/activate
pip install -e .
. venv/bin/activate
pip install plotly-geo
- run:
name: Test core
command: |
cd packages/python/plotly
. venv/bin/activate
pytest plotly/tests/test_core
no_output_timeout: 20m
- run:
name: Test optional
command: |
cd packages/python/plotly
. venv/bin/activate
pytest plotly/tests/test_optional
no_output_timeout: 40m
- run:
name: Test utils
command: |
cd packages/python/plotly
. venv/bin/activate
pytest _plotly_utils/tests/
no_output_timeout: 20m
- run:
name: Test io
command: |
cd packages/python/plotly
. venv/bin/activate
pytest plotly/tests/test_io
no_output_timeout: 20m
- run:
name: Test dependencdies not imported
command: |
cd packages/python/plotly
. venv/bin/activate
pytest -x test_init/test_dependencies_not_imported.py
- run:
name: Test lazy imports
command: |
cd packages/python/plotly
. venv/bin/activate
pytest -x test_init/test_lazy_imports.py
test_orca:
Expand All @@ -107,17 +97,15 @@ commands:
- run:
name: Install dependencies
command: |
cd packages/python/plotly
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
pip install -r ./test_requirements/requirements_<<parameters.py>>_optional.txt
- run:
name: Install plotly-geo
command: |
cd packages/python/plotly-geo
. ../plotly/venv/bin/activate
pip install -e .
. venv/bin/activate
pip install plotly-geo
- run:
name: Install orca
command: |
Expand All @@ -129,12 +117,11 @@ commands:
- run:
name: Test orca
command: |
cd packages/python/plotly
. venv/bin/activate
pytest plotly/tests/test_orca
no_output_timeout: 20m
- store_artifacts:
path: packages/python/plotly/plotly/tests/test_orca/images/linux/failed
path: plotly/tests/test_orca/images/linux/failed

jobs:
check-code-formatting:
Expand Down Expand Up @@ -283,9 +270,10 @@ jobs:
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
pip install -e ./packages/python/plotly
pip install -e ./packages/python/plotly-geo
pip install -r ./packages/python/plotly/test_requirements/requirements_39_pandas_2_optional.txt
pip install -e .
pip install plotly-geo
pip install -r test_requirements/requirements_39_pandas_2_optional.txt

- run:
name: Build html figures (Pandas 2)
command: |
Expand All @@ -309,30 +297,6 @@ jobs:
npx percy snapshot -c test/percy/snapshots.yml test/percy/
rm test/percy/*.html

# Chart studio
python_38_chart_studio:
docker:
- image: cimg/python:3.8
resource_class: large

steps:
- checkout
- run:
name: Install dependencies
command: |
cd packages/python/chart-studio
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
pip install -r ./test_requirements/requirements_38.txt
- run:
name: Tests
command: |
cd packages/python/chart-studio
. venv/bin/activate
pytest -x chart_studio/tests/
no_output_timeout: 20m

plotlyjs_dev_build:
docker:
- image: cimg/python:3.8-node
Expand All @@ -345,7 +309,6 @@ jobs:
- run:
name: Install dependencies
command: |
cd packages/python/plotly
python -m venv venv
. venv/bin/activate
pip install --upgrade pip wheel
Expand All @@ -354,27 +317,23 @@ jobs:
- run:
name: Update jupyter widget plotly.js version
command: |
cd packages/python/plotly
. venv/bin/activate
python setup.py updateplotlywidgetversion
- run:
name: Update plotly.js to dev
command: |
cd packages/python/plotly
. venv/bin/activate
python setup.py updateplotlyjsdev
- run:
name: Test core
command: |
cd packages/python/plotly
. venv/bin/activate
locale
pytest -k 'not nodev' plotly/tests/test_core
no_output_timeout: 20m
- run:
name: Commit
command: |
cd packages/python/plotly
git config --global user.email "[email protected]"
git config --global user.name "Your Name"
git add -A
Expand All @@ -383,12 +342,11 @@ jobs:
- run:
name: Build source distribution packages
command: |
cd packages/python/plotly
. venv/bin/activate
python setup.py sdist
when: always
- store_artifacts:
path: packages/python/plotly/dist
path: dist/

full_build:
docker:
Expand All @@ -415,7 +373,7 @@ jobs:
command: |
eval "$(conda shell.bash hook)"
conda activate env
cd packages/python/plotly/js
cd js
npm ci
npm run build
git status
Expand All @@ -425,27 +383,25 @@ jobs:
command: |
eval "$(conda shell.bash hook)"
conda activate env
cd packages/python/plotly
python setup.py sdist bdist_wheel
cp -R dist ../../../output/dist
cp -R dist output/dist
git status

- run:
name: Conda Build
command: |
eval "$(conda shell.bash hook)"
conda activate env
cd packages/python/plotly
conda build recipe/
mv /opt/conda/envs/env/conda-bld/noarch/plotly*.tar.bz2 ../../../output
mv /opt/conda/envs/env/conda-bld/noarch/plotly*.tar.bz2 output/
git status

- run:
name: Build Widget javascript bundle
command: |
eval "$(conda shell.bash hook)"
conda activate env
cd packages/python/plotly/js
cd js
npm ci
npm run build

Expand Down Expand Up @@ -494,12 +450,12 @@ jobs:
pip install -r requirements.txt
if [ "${CIRCLE_BRANCH}" != "doc-prod" ]; then
pip uninstall -y plotly
cd ../packages/python/plotly
cd ..
pip install -e .
cd js
npm ci
npm run build
cd ../../../../doc
cd ../doc
fi
cd ..
- run:
Expand Down Expand Up @@ -576,9 +532,9 @@ jobs:
# graph_objs/graph_objects
if [ "${CIRCLE_BRANCH}" == "doc-prod" ]; then
pip uninstall -y plotly
cd ../packages/python/plotly
cd ..
pip install -e .
cd ../../../doc
cd doc
cd apidoc
make html
cd _build/html
Expand Down
2 changes: 0 additions & 2 deletions .flake8

This file was deleted.

2 changes: 1 addition & 1 deletion .gitattributes
Original file line number Diff line number Diff line change
@@ -1 +1 @@
packages/python/plotly/plotly/_version.py export-subst
plotly/_version.py export-subst
10 changes: 5 additions & 5 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
0
0.html
iframe_figures/
packages/python/plotly/plotly/tests/test_orca/images/linux/failed/
plotly/tests/test_orca/images/linux/failed/

*.egg-info

Expand All @@ -15,6 +15,7 @@ doc/python/raw.githubusercontent.com/

# Don't ignore dataset files
!*.csv.gz
!*.geojson.gz

*.ipynb

Expand Down Expand Up @@ -59,10 +60,9 @@ doc/.ipynb_checkpoints
tags
doc/check-or-enforce-order.py

packages/javascript/jupyterlab-plotly/lib/
packages/python/plotly/jupyterlab_plotly/labextension/
packages/python/plotly/jupyterlab_plotly/nbextension/index.js*
packages/python/plotly/plotly/package_data/widgetbundle.js
jupyterlab_plotly/labextension/
jupyterlab_plotly/nbextension/index.js*
plotly/package_data/widgetbundle.js

test/percy/*.html
test/percy/pandas2/*.html
6 changes: 0 additions & 6 deletions .pre-commit-config.yaml

This file was deleted.

File renamed without changes.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,8 @@ or conda
conda install -c plotly plotly-geo=1.0.0
```

`plotly-geo` can be found on Github at https://github.com/plotly/plotly-geo.

## Copyright and Licenses

Code and documentation copyright 2019 Plotly, Inc.
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ def perform_codegen():
# ---------------------------------
# (relative to project root)
abs_file_path = opath.realpath(__file__)
packages_py = opath.dirname(opath.dirname(opath.dirname(abs_file_path)))
outdir = opath.join(packages_py, "plotly", "plotly")
project_root = opath.dirname(opath.dirname(abs_file_path))
outdir = opath.join(project_root, "plotly")

# Delete prior codegen output
# ---------------------------
Expand All @@ -113,7 +113,7 @@ def perform_codegen():
# Load plotly schema
# ------------------
plot_schema_path = opath.join(
packages_py, "plotly", "codegen", "resources", "plot-schema.json"
project_root, "codegen", "resources", "plot-schema.json"
)

with open(plot_schema_path, "r") as f:
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Loading