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

Collect common links under custom file #82

Closed
wants to merge 4 commits into from
Closed

Conversation

jorgepiloto
Copy link
Member

Resolves #70.

@MaxJPRey MaxJPRey self-requested a review April 19, 2022 18:48
@jorgepiloto jorgepiloto force-pushed the feat/links branch 5 times, most recently from f642329 to 58e7b72 Compare April 20, 2022 11:32
.. _PyAEDT: https://github.com/pyansys/pyaedt
.. _PyMAPDL: https://github.com/pyansys/pymapdl
.. _PyAnsys: https://docs.pyansys.com/
.. _PyAnsys PyPI: https://pkgs.dev.azure.com/pyansys/_packaging/pyansys/pypi
Copy link
Member Author

Choose a reason for hiding this comment

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

This link does no longer work.

@jorgepiloto jorgepiloto force-pushed the feat/links branch 3 times, most recently from 8f9bab5 to a6b67c5 Compare April 20, 2022 18:09
@jorgepiloto jorgepiloto force-pushed the feat/links branch 2 times, most recently from 4cc82cb to fdefe74 Compare May 3, 2022 12:45
@jorgepiloto jorgepiloto force-pushed the feat/links branch 3 times, most recently from cd1fa2b to d99348e Compare May 3, 2022 15:47
@jorgepiloto
Copy link
Member Author

The error in the PDF building process is caused by this PR, as the nightly build ran without problems. Reading the logs locally to identify the problem...

@jorgepiloto
Copy link
Member Author

This is likely to be introduced by some "infinite-loop" when solving links. This works fine in the HTML builder but not sure if will do so for the PDF.

@jorgepiloto
Copy link
Member Author

This guide might be an alternate approach to this: https://sublime-and-sphinx-guide.readthedocs.io/en/latest/references.html#setup-external-links

Comment on lines +109 to +110
placing them into the ``vars`` list, all while following the Style
Guide for Python Code, defined in `PEP 8`_.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
placing them into the ``vars`` list, all while following the Style
Guide for Python Code, defined in `PEP 8`_.
placing them into the ``vars`` list, all while following the style
guidelines for Python code that are defined in `PEP 8`_.

@@ -3,10 +3,10 @@ Data Transfer

Abstracted APIs should attempt to hide the implementation details of
the remote or local API in a well organized data model. This data
model should avoid returning raw JSON files, gRPC messages, SWIG objects,
model should avoid returning raw JSON files, gRPC messages, `SWIG`_ objects,
or .NET objects. It should preferably return standard Python objects
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
or .NET objects. It should preferably return standard Python objects
and .NET objects. It should preferably return standard Python objects

Comment on lines +13 to +17
Consider the following code examples. The left-hand side shows the amount of
work to start, establish a connection to, and submit an input file to MAPDL
using auto-generated gRPC interface files. For more information, see
`pyansys-protos-generator`_. The right-hand side shows the same workflow but
uses the `PyMAPDL`_ library.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Consider the following code examples. The left-hand side shows the amount of
work to start, establish a connection to, and submit an input file to MAPDL
using auto-generated gRPC interface files. For more information, see
`pyansys-protos-generator`_. The right-hand side shows the same workflow but
uses the `PyMAPDL`_ library.
Consider the following code examples. The left side shows the amount of
work to start, establish a connection to, and submit an input file to MAPDL
using auto-generated gRPC interface files. For more information, see
`pyansys-protos-generator`_. The right side shows the same workflow but
uses the `PyMAPDL`_ library.

@@ -1,15 +1,15 @@
Beyond PEP8
###########
===========
This topic describes any delineations, clarifications, or additional procedures above and
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This topic describes any delineations, clarifications, or additional procedures above and
This topic describes any delineations, clarifications, or additional procedures

outlines deprecation best practices and a `Deprecation library <https://deprecation.readthedocs.io/>`_
already exists. However, there is no official guidance regarding deprecating features
in an API within Python. This page seeks to clarify this issue and others.
For example, `Stack Overflow Answer`_ outlines deprecation best practices and a
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
For example, `Stack Overflow Answer`_ outlines deprecation best practices and a
For example, `Stack Overflow Answer`_ outlines deprecation best practices, and a

open-source nature of the project, documentation built by each PyAnsys library
or project should be public and editable by users and the community at large.

After following the documentation build strategy contained within both the
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
After following the documentation build strategy contained within both the
After following the documentation build strategy contained within both

.. _pandas: https://pandas.pydata.org/
.. _pyvista: https://docs.pyvista.org/
.. _scipy: https://scipy.org/
.. _tensorflow:
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this is empty?? @jorgepiloto

@PipKat
Copy link
Member

PipKat commented Mar 13, 2024

@jorgepiloto and @RobPasMue In draft PR #410, I moved all links with explicit target names to a central link file, I believe that we could close this PR.)

@RobPasMue
Copy link
Member

Let's close it yeah - thanks @PipKat - I'll review that one =)

@RobPasMue RobPasMue closed this Mar 14, 2024
@RobPasMue RobPasMue deleted the feat/links branch March 14, 2024 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collect usual hyperlinks in a common links.rst
5 participants