Skip to content

Commit f06ed71

Browse files
committed
Merge with master
2 parents ef4abb8 + 82736ad commit f06ed71

36 files changed

+46
-167
lines changed

.circleci/config.yml

Lines changed: 35 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ commands:
2424
name: Test core
2525
command: |
2626
. venv/bin/activate
27-
pytest tests/test_core
27+
python -m pytest tests/test_core
2828
no_output_timeout: 20m
2929

3030
test_optional:
@@ -39,46 +39,47 @@ commands:
3939
- run:
4040
name: Install dependencies
4141
command: |
42-
python -m venv venv
43-
. venv/bin/activate
44-
pip install -e .
45-
pip install -r ./test_requirements/requirements_optional.txt
42+
curl -LsSf https://astral.sh/uv/install.sh | sh
43+
uv venv
44+
source .venv/bin/activate
45+
uv pip install .
46+
uv pip install -r ./test_requirements/requirements_optional.txt
4647
cd js
4748
npm ci
4849
npm run build
4950
- run:
5051
name: Test core
5152
command: |
52-
. venv/bin/activate
53-
pytest tests/test_core
53+
source .venv/bin/activate
54+
python -m pytest tests/test_core
5455
no_output_timeout: 20m
5556
- run:
5657
name: Test optional
5758
command: |
58-
. venv/bin/activate
59-
pytest tests/test_optional
59+
source .venv/bin/activate
60+
python -m pytest tests/test_optional
6061
no_output_timeout: 40m
6162
- run:
6263
name: Test utils
6364
command: |
64-
. venv/bin/activate
65-
pytest tests/test_plotly_utils/
65+
source .venv/bin/activate
66+
python -m pytest tests/test_plotly_utils/
6667
no_output_timeout: 20m
6768
- run:
6869
name: Test io
6970
command: |
70-
. venv/bin/activate
71-
pytest tests/test_io
71+
source .venv/bin/activate
72+
python -m pytest tests/test_io
7273
no_output_timeout: 20m
7374
- run:
74-
name: Test dependencdies not imported
75+
name: Test dependencies not imported
7576
command: |
76-
. venv/bin/activate
77+
source .venv/bin/activate
7778
python -m pytest -x test_init/test_dependencies_not_imported.py
7879
- run:
7980
name: Test lazy imports
8081
command: |
81-
. venv/bin/activate
82+
source .venv/bin/activate
8283
python -m pytest -x test_init/test_lazy_imports.py
8384
8485
jobs:
@@ -235,24 +236,24 @@ jobs:
235236
name: Build html figures (Pandas 2)
236237
command: |
237238
. venv/bin/activate
238-
python test/percy/plotly-express.py
239+
python tests/percy/plotly-express.py
239240
- run:
240241
name: Build html figures (Pandas 1) and compare
241242
command: |
242243
. venv/bin/activate
243-
mkdir test/percy/pandas2
244-
mv test/percy/*.html test/percy/pandas2/
244+
mkdir tests/percy/pandas2
245+
mv tests/percy/*.html tests/percy/pandas2/
245246
# 1.1 is the earliest minor with Py3.9 wheels
246247
pip install "pandas==1.1.5"
247-
python test/percy/plotly-express.py
248-
python test/percy/compare-pandas.py
249-
rm -rf test/percy/pandas2
248+
python tests/percy/plotly-express.py
249+
python tests/percy/compare-pandas.py
250+
rm -rf tests/percy/pandas2
250251
- run:
251252
name: Run percy snapshots
252253
command: |
253254
npm i @percy/cli
254-
npx percy snapshot -c test/percy/snapshots.yml test/percy/
255-
rm test/percy/*.html
255+
npx percy snapshot -c tests/percy/snapshots.yml tests/percy/
256+
rm tests/percy/*.html
256257
257258
plotlyjs_dev_build:
258259
docker:
@@ -351,43 +352,33 @@ jobs:
351352
- "dc:5f:39:48:00:b4:72:34:e1:d2:c4:e1:1f:d1:e2:ce" #plotlydocbot
352353

353354
- checkout
354-
355-
# Download and cache dependencies
356-
- restore_cache:
357-
key: v2-dependencies-{{ checksum "doc/requirements.txt" }}-{{ checksum ".circleci/config.yml" }}
358355
- browser-tools/install-chrome
359356
- browser-tools/install-chromedriver
360357

361358
- run:
362359
name: install dependencies
363360
command: |
364361
cd doc
365-
python -m venv venv
366-
. venv/bin/activate
367-
pip uninstall -y plotly
368-
pip install -r requirements.txt
362+
curl -LsSf https://astral.sh/uv/install.sh | sh
363+
uv venv
364+
source .venv/bin/activate
365+
uv pip install -r requirements.txt
369366
if [ "${CIRCLE_BRANCH}" != "doc-prod" ]; then
370-
pip uninstall -y plotly
367+
uv pip uninstall plotly
371368
cd ..
372-
pip install -e .
369+
uv pip install -e .
373370
cd js
374371
npm ci
375372
npm run build
376373
cd ../doc
377374
fi
378375
cd ..
379376
380-
- save_cache:
381-
paths:
382-
- ./doc/venv
383-
- ./doc/node_modules
384-
key: v1-dependencies-{{ checksum "doc/requirements.txt" }}-{{ checksum ".circleci/config.yml" }}
385-
386377
- run:
387378
name: make html
388379
command: |
389380
cd doc
390-
. venv/bin/activate
381+
source .venv/bin/activate
391382
echo ${mapbox_token} > python/.mapbox_token
392383
make -kj8 || make -kj8
393384
curl https://raw.githubusercontent.com/plotly/graphing-library-docs/master/front-matter-ci.py > front-matter-ci.py
@@ -436,14 +427,14 @@ jobs:
436427
name: make doc
437428
command: |
438429
cd doc
439-
. venv/bin/activate
430+
source .venv/bin/activate
440431
# For the API doc, we need to use the local version of plotly
441432
# since we are tweaking the source because of
442433
# graph_objs/graph_objects
443434
if [ "${CIRCLE_BRANCH}" == "doc-prod" ]; then
444-
pip uninstall -y plotly
435+
uv pip uninstall plotly
445436
cd ..
446-
pip install -e .
437+
uv pip install -e .
447438
cd doc
448439
cd apidoc
449440
make html

.github/pull_request_template.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ below :-).
2727
2828
## Code PR
2929
30-
- [ ] I have read through the [contributing notes](https://github.com/plotly/plotly.py/blob/master/contributing.md) and understand the structure of the package. In particular, if my PR modifies code of `plotly.graph_objects`, my modifications concern the `codegen` files and not generated files.
30+
- [ ] I have read through the [contributing notes](https://github.com/plotly/plotly.py/blob/master/CONTRIBUTING.md) and understand the structure of the package. In particular, if my PR modifies code of `plotly.graph_objects`, my modifications concern the `codegen` files and not generated files.
3131
- [ ] I have added tests (if submitting a new feature or correcting a bug) or
3232
modified existing tests.
3333
- [ ] For a new feature, I have added documentation examples in an existing or

.gitignore

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,5 +61,5 @@ tags
6161
doc/check-or-enforce-order.py
6262
plotly/package_data/widgetbundle.js
6363

64-
test/percy/*.html
65-
test/percy/pandas2/*.html
64+
tests/percy/*.html
65+
tests/percy/pandas2/*.html

CHANGELOG.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -716,7 +716,7 @@ This version includes several performance improvements ([#2368](https://github.c
716716
- Introduce range breaks on date axes (for example, to remove week-ends) via `layout.xaxis.rangebreaks`
717717
- Introduce a new unified x (or y) hovermode (`layout.hovermode="x unified"`), in which the hover box shows the information for all traces at a given x (or y) position
718718
- Add `node.customdata` and `link.customdata` to sankey traces
719-
- Updated [contributing notes](https://github.com/plotly/plotly.py/blob/master/contributing.md) for more explanations on how to contribute to plotly.py [#2290](https://github.com/plotly/plotly.py/pull/2290). Please give feedback on these notes!
719+
- Updated [contributing notes](https://github.com/plotly/plotly.py/blob/master/CONTRIBUTING.md) for more explanations on how to contribute to plotly.py [#2290](https://github.com/plotly/plotly.py/pull/2290). Please give feedback on these notes!
720720
- Updated documentation examples [#2325](https://github.com/plotly/plotly.py/pull/2325), and to show how to color links in Sankey diagrams [#2291](https://github.com/plotly/plotly.py/pull/2291).
721721
- Special thanks to [@SylwiaOliwia2](https://github.com/SylwiaOliwia2) and [@dangercrow](https://github.com/dangercrow) for improving our documentation!
722722

@@ -1738,7 +1738,7 @@ This is a major version with many exciting updates. See the [Introducing plotly.
17381738
- Error message for `plotly.figure_factory.create_choropleth` is now helpful to Anaconda users who do not have the correct modules installed for the County Choropleth figure factory.
17391739

17401740
### Changed / Deprecated
1741-
Please see the [migration guid](migration-guide.md) for a full list of the changes and deprecations in version 3.0.0
1741+
Please see the [migration guide](MIGRATION_GUIDE.md) for a full list of the changes and deprecations in version 3.0.0
17421742

17431743

17441744

File renamed without changes.
File renamed without changes.

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ Built on top of [plotly.js](https://github.com/plotly/plotly.js), `plotly.py` is
6767
---
6868

6969
- [Online Documentation](https://plotly.com/python/)
70-
- [Contributing to plotly](https://github.com/plotly/plotly.py/blob/master/contributing.md)
70+
- [Contributing to plotly](https://github.com/plotly/plotly.py/blob/master/CONTRIBUTING.md)
7171
- [Changelog](https://github.com/plotly/plotly.py/blob/master/CHANGELOG.md)
7272
- [Code of Conduct](https://github.com/plotly/plotly.py/blob/master/CODE_OF_CONDUCT.md)
7373
- [Community forum](https://community.plotly.com)

release.md renamed to RELEASE.md

File renamed without changes.

_plotly_future_/__init__.py

Lines changed: 0 additions & 78 deletions
This file was deleted.

_plotly_future_/extract_chart_studio.py

Whitespace-only changes.

0 commit comments

Comments
 (0)