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

docs + refactor: Sphinx extensions added + Documentation updated + Docstrings updated. #8

Merged
merged 12 commits into from
Jan 1, 2023

Conversation

drageelr
Copy link
Owner

@drageelr drageelr commented Jan 1, 2023

Overview: What does this pull request change?

  • Following sphinx extensions are added and configured:
    • intersphinx: Configured for python, manim and numpy.
    • napolean
    • viewcode
    • sphinx_autodoc_typehints
  • Created a substitution reference file refsub.rst for code element references.
  • Updated "Animating Arrays" guide in accordance with refsub.rst.
  • Docstrings for the following classes updated:
    • MArrayElement
    • MArray
    • MArrayPointer
    • MVariable
    • MArrayElementComp
    • MArrayDirection

Motivation and Explanation: Why and how do your changes improve the library?

  • References to other library documentations is enabled.
  • Aggregated references used in the documentation in a single file and utilized them in other doc pages to reduce clutter.
  • Simplified and unified docstrings for all classes.

Further Information and Comments

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section

@drageelr drageelr added documentation Improvements or additions to documentation refactor Code change that neither fixes a bug nor adds a new feature labels Jan 1, 2023
@drageelr drageelr self-assigned this Jan 1, 2023
@drageelr drageelr merged commit bd560e1 into dev Jan 1, 2023
@drageelr drageelr deleted the drageelr-dev branch January 1, 2023 10:27
drageelr added a commit that referenced this pull request Jan 8, 2023
* docs + refactor: Sphinx extensions added + Documentation updated + Docstrings updated. (#8)

* docs: added intersphinx mapping + add_module_names config updated

* docs: fixed references in Animating Arrays guide

* docs: viewcode + napolean + autodoc_typehints sphinx extensions added

* refactor: updated docstrings for MVariable

* docs: changed func domain to meth in Animating Arrays guide

* refactor: updated docstrings for MArrayElement

* refactor: fixed Scene.play references in MVariable & MArrayElement docstrings + updated .flake8 to ignore error

* refactor: docstrings for typing.Union fixed in MVariable + updated .flake8 to ignore error

* refactor: updated docstrings for MArray + minor edit to MArrayElement docstrings

* Updated docstrings for all methods of MArray.
* Updated __init__ and class docstring of MArrayElement.
* Renamed MArray.fetch_arr_label() to MArray.fetch_mob_arr_label().
* Placed play_anim and play_anim_args parameter at the end of MArray.append_elem().

* refactor: updated docstrings for MArrayElementComp & MArrayDirection

* docs: created refsub.rst + added substitutions in Animating Arrays guide

* Created refsub.rst file that contains references to code elements.
* Updated Animating Arrays guide with substitutions from refsub.rst.

* refactor: updated docstrings for MArrayPointer

* feat: MArraySlidingWindow added + docs updated (#9)

* feat: MArraySlidingWindow added

* docs: MArraySlidingWindow docs added

Example Gallery
Animating Arrays Guide
Reference Manual

* docs: updated changelog for release 0.1.7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation refactor Code change that neither fixes a bug nor adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant