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

Autolev parser docs 3 #15067

Merged
merged 10 commits into from Aug 13, 2018

Conversation

Projects
None yet
5 participants
@NikhilPappu
Contributor

NikhilPappu commented Aug 9, 2018

  • other
    • Added the issues and future improvements sections to autolev_parser.rst in the mechanics part of the documentation.

NikhilPappu added some commits Aug 6, 2018

Added documentation for Autolev Parser under Mechanics
This PR contains the Introduction and Usage sections
@sympy-bot

This comment has been minimized.

Show comment
Hide comment
@sympy-bot

sympy-bot Aug 9, 2018

Hi, I am the SymPy bot (v127). I'm here to make sure this pull request has a release notes entry. Please read the guide on how to write release notes.

Click here to see the pull request description that was parsed.

<!-- BEGIN RELEASE NOTES -->
- other
  - Added the issues and future improvements sections to autolev_parser.rst in the mechanics part of the documentation.
<!-- END RELEASE NOTES -->

Your release notes are in good order.

Here is what the release notes will look like:

  • other
    • Added the issues and future improvements sections to autolev_parser.rst in the mechanics part of the documentation. (#15067 by @NikhilPappu)

This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.2.1.

Note: This comment will be updated with the latest check if you edit the pull request. You need to reload the page to see it.

Update

The release notes on the wiki have been updated.

sympy-bot commented Aug 9, 2018

Hi, I am the SymPy bot (v127). I'm here to make sure this pull request has a release notes entry. Please read the guide on how to write release notes.

Click here to see the pull request description that was parsed.

<!-- BEGIN RELEASE NOTES -->
- other
  - Added the issues and future improvements sections to autolev_parser.rst in the mechanics part of the documentation.
<!-- END RELEASE NOTES -->

Your release notes are in good order.

Here is what the release notes will look like:

  • other
    • Added the issues and future improvements sections to autolev_parser.rst in the mechanics part of the documentation. (#15067 by @NikhilPappu)

This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.2.1.

Note: This comment will be updated with the latest check if you edit the pull request. You need to reload the page to see it.

Update

The release notes on the wiki have been updated.

NikhilPappu added some commits Aug 11, 2018

@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Aug 13, 2018

Member

@NikhilPappu is this ready to be merged? It looks good to me.

Member

certik commented Aug 13, 2018

@NikhilPappu is this ready to be merged? It looks good to me.

@@ -662,6 +662,7 @@ def _doctest(*paths, **kwargs):
split = kwargs.get('split', None)
blacklist.extend([
"doc/src/modules/plotting.rst", # generates live plots
"doc/src/modules/physics/mechanics/autolev_parser.rst",

This comment has been minimized.

@certik

certik Aug 13, 2018

Member

Why is this blacklisted?

@certik

certik Aug 13, 2018

Member

Why is this blacklisted?

This comment has been minimized.

@NikhilPappu

NikhilPappu Aug 13, 2018

Contributor

The doctest in autolev_parser.rst is not meant to be executed. It just shows the user what commands should be executed in the shell.

@NikhilPappu

NikhilPappu Aug 13, 2018

Contributor

The doctest in autolev_parser.rst is not meant to be executed. It just shows the user what commands should be executed in the shell.

@NikhilPappu

This comment has been minimized.

Show comment
Hide comment
@NikhilPappu

NikhilPappu Aug 13, 2018

Contributor

@certik I think it is ready to be merged.

Contributor

NikhilPappu commented Aug 13, 2018

@certik I think it is ready to be merged.

@certik certik merged commit 2961d8f into sympy:master Aug 13, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
sympy-bot/release-notes The release notes look OK
Details
@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Aug 13, 2018

Member

Thanks!

Member

certik commented Aug 13, 2018

Thanks!

.. code-block:: python
SymPy Code
----------

This comment has been minimized.

@moorepants

moorepants Aug 14, 2018

Member

These should be made comments with #.

@moorepants

moorepants Aug 14, 2018

Member

These should be made comments with #.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment