Skip to content

Replace recommonmark with MyST, fix all Sphinx issues #72

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

Merged
merged 1 commit into from
Nov 9, 2020
Merged

Conversation

rgommers
Copy link
Member

@rgommers rgommers commented Nov 9, 2020

No description provided.

@rgommers
Copy link
Member Author

rgommers commented Nov 9, 2020

This is all just rendering/formatting fixes, so I'll merge it straight away.

@rgommers rgommers merged commit 95809a6 into master Nov 9, 2020
@rgommers rgommers deleted the myst branch November 9, 2020 23:06
@@ -7,6 +7,8 @@ A conforming implementation of the array API standard must provide and support t
- Positional parameters must be [positional-only](https://www.python.org/dev/peps/pep-0570/) parameters. Positional-only parameters have no externally-usable name. When a function accepting positional-only parameters is called, positional arguments are mapped to these parameters based solely on their order.
- Optional parameters must be [keyword-only](https://www.python.org/dev/peps/pep-3102/) arguments.

## Objects in API

<!-- NOTE: please keep the functions in alphabetical order -->

### <a name="arange" href="#arange">#</a> arange(start, /, *, stop=None, step=1, dtype=None)
Copy link
Member

Choose a reason for hiding this comment

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

Some headers like this were modified and others were not. Was this just an omission?

It would also be helpful if all the function stub headers had some way to identify them for my regex that extracts them. I think a simple thing would be to add a cross reference target for each one, like

(function-arange)=
### arange(start, /, *, stop=None, step=1, dtype=None)

or

(constant-pi)=
### pi

The cross reference targets would be useful to have anyway.. I can make a PR adding this if you agree (it shouldn't affect the final rendering).

Copy link
Member

Choose a reason for hiding this comment

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

Actually, I think this would affect the anchor names (they would be like #function-arange instead of #arange). Is this acceptable, or should I use some other way to mark function definitions?

Copy link
Member Author

Choose a reason for hiding this comment

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

Some headers like this were modified and others were not. Was this just an omission?

I just fixed the ones that were used as references. There are zero Sphinx warnings, and we should keep it this way. Long-term this needs a bigger overhaul, using signatures as h3 headings doesn't work all that well. That's why I didn't bother updating everything.

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually, I think this would affect the anchor names (they would be like #function-arange instead of #arange). Is this acceptable, or should I use some other way to mark function definitions?

Anchor names are a bit of a pain now, could be useful to unify. I like your proposed scheme.

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.

2 participants