Merge pull request #5002 from plotly/simplify-dir-structure

Move `packages/python/plotly` to top level

Author: emilykl

.circleci/config.yml

@@ -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
@@ -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
@@ -52,47 +49,40 @@ commands:
       - run:
           name: Install plotly-geo
           command: |
-            cd packages/python/plotly
             . 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:
@@ -107,15 +97,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>>_optional.txt
       - run:
           name: Install plotly-geo
           command: |
-            cd packages/python/plotly
             . venv/bin/activate
             pip install plotly-geo
       - run:
@@ -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:
@@ -283,9 +270,9 @@ jobs:
             python -m venv venv
             . venv/bin/activate
             pip install --upgrade pip wheel
-            pip install -e ./packages/python/plotly
+            pip install -e .
             pip install plotly-geo
-            pip install -r ./packages/python/plotly/test_requirements/requirements_39_pandas_2_optional.txt
+            pip install -r test_requirements/requirements_39_pandas_2_optional.txt
 
       - run:
           name: Build html figures (Pandas 2)
@@ -322,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
@@ -331,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
@@ -360,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:
@@ -392,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
@@ -402,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
 
@@ -471,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:
@@ -553,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

.gitattributes

@@ -1 +1 @@
-packages/python/plotly/plotly/_version.py export-subst
+plotly/_version.py export-subst

.gitignore

@@ -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
 
@@ -15,6 +15,7 @@ doc/python/raw.githubusercontent.com/
 
 # Don't ignore dataset files
 !*.csv.gz
+!*.geojson.gz
 
 *.ipynb
 
@@ -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

packages/python/plotly/MANIFEST.in MANIFEST.in

@@ -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
     # ---------------------------
@@ -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: