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

Apigen recursive #2243

Merged
merged 14 commits into from
Mar 11, 2020
  •  
  •  
  •  
3 changes: 2 additions & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -409,7 +409,7 @@ jobs:
cd ../../../doc
fi
echo 'export PATH="/home/circleci/project/doc/node_modules/.bin:$PATH"' >> $BASH_ENV
sudo apt-get install -y poppler-utils libxtst6 xvfb libgtk2.0-0 libgconf-2-4 libnss3 libasound2
sudo apt-get install -y poppler-utils libxtst6 xvfb libgtk2.0-0 libgconf-2-4 libnss3 libasound2 rename
cd ..

- save_cache:
Expand Down Expand Up @@ -472,6 +472,7 @@ jobs:
git config user.name plotlydocbot
git config user.email accounts@plot.ly
git add *
git add *
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redundant?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hum let me fix this

git add .nojekyll
git commit -m "build of https://github.com/plotly/plotly.py/commit/${CIRCLE_SHA1}"
git push --force git@github.com:plotly/plotly.py-docs.git master:gh-pages
Expand Down
5 changes: 5 additions & 0 deletions doc/apidoc/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,10 @@ help:

# Catch-all target: route all unknown targets to Sphinx using the new
# "make mode" option. $(O) is meant as a shortcut for $(SPHINXOPTS).
# For sphinx-apidoc the first positional path is the module to document
# then all the other ones are paths to exclude for the doc generation
%: Makefile
sphinx-apidoc -o generated ../../packages/python/plotly/plotly ../../packages/python/plotly/plotly/validators ../../packages/python/plotly/plotly/tests ../../packages/python/plotly/plotly/matplotlylib/ ../../packages/python/plotly/plotly/offline ../../packages/python/plotly/plotly/api
@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
rename 's/graph_objs/graph_objects/' _build/html/*.html _build/html/generated/*.html
sed -i 's/graph_objs/graph_objects/g' _build/html/*.html _build/html/generated/*.html
15 changes: 0 additions & 15 deletions doc/apidoc/_templates/class.rst

This file was deleted.

1 change: 1 addition & 0 deletions doc/apidoc/_templates/trace.rst
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@

.. automodule:: plotly.graph_objects.{{ objname.lower() }}
:members:
:undoc-members:


.. raw:: html
Expand Down
3 changes: 1 addition & 2 deletions doc/apidoc/plotly.graph_objects.rst
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
`plotly.graph_objects`: low-level interface to figures, traces and layout
=========================================

.. currentmodule:: plotly.graph_objects
.. currentmodule:: plotly.graph_objs

:mod:`plotly.graph_objects` contains the building blocks of plotly :class:`Figure`: traces
(:class:`Scatter`, :class:`Bar`, ...) and :class:`Layout`
Expand Down Expand Up @@ -31,7 +31,6 @@ Layout
Layout



Simple charts
--------------

Expand Down
4 changes: 2 additions & 2 deletions packages/javascript/plotlywidget/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/python/plotly/_plotly_utils/basevalidators.py
Original file line number Diff line number Diff line change
Expand Up @@ -2421,7 +2421,7 @@ def description(self):
"""\
The '{plotly_name}' property is an instance of {class_str}
that may be specified as:
- An instance of {module_str}.{class_str}
- An instance of :class:`{module_str}.{class_str}`
- A dict of string/value properties that will be passed
to the {class_str} constructor

Expand Down
4 changes: 2 additions & 2 deletions packages/python/plotly/codegen/datatypes.py
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ def __init__(self"""
(
f"arg",
f"dict of properties compatible with this constructor "
f"or an instance of {class_name}",
f"or an instance of :class:`{class_name}`",
)
]

Expand Down Expand Up @@ -321,7 +321,7 @@ def __init__(self"""
raise ValueError(\"\"\"\\
The first argument to the {class_name}
constructor must be a dict or
an instance of {class_name}\"\"\")
an instance of :class:`{class_name}`\"\"\")

# Handle skip_invalid
# -------------------
Expand Down
2 changes: 1 addition & 1 deletion packages/python/plotly/codegen/figure.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ class {fig_classname}({base_classname}):\n"""
def __init__(self, data=None, layout=None,
frames=None, skip_invalid=False, **kwargs):
\"\"\"
Create a new {fig_classname} instance
Create a new :class:{fig_classname} instance

Parameters
----------
Expand Down
4 changes: 2 additions & 2 deletions packages/python/plotly/codegen/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -941,7 +941,7 @@ def get_constructor_params_docstring(self, indent=12):
f"{subtype_node.name_datatype_class}"
)
subtype_description = (
f"A tuple of {class_name} instances or "
f"A tuple of :class:`{class_name}` instances or "
"dicts with compatible properties"
)
elif subtype_node.is_compound:
Expand All @@ -958,7 +958,7 @@ def get_constructor_params_docstring(self, indent=12):
)

subtype_description = (
f"{class_name} instance or dict with compatible properties"
f":class:`{class_name}` instance or dict with compatible properties"
)
else:
subtype_description = ""
Expand Down
2 changes: 1 addition & 1 deletion packages/python/plotly/plotly/express/_doc.py
Original file line number Diff line number Diff line change
Expand Up @@ -536,5 +536,5 @@ def make_docstring(fn, override_dict={}):
param_type = param_doc[0]
result += "%s: %s\n%s\n" % (param, param_type, param_desc)
result += "\nReturns\n-------\n"
result += " A `Figure` object."
result += " plotly.graph_objects.Figure"
return result
Loading