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
DOC: update OptimizeResult notes #14693
Conversation
Closes: scipy#14683 Does two things: - Address the problem of `Notes` being placed before `Attributes` - Edit the spelling in the first sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OptimizeResult
at least needs to be in single backticks. Not sure what should be done with keys
but I think this will link to the documentation.
[skip azp] [skip actions]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mdhaber It's sphinx, so we will have to use rst instead of markdown here I believe.
Edit: ... or not? This is a bit surprising, but also pretty cool. I'll have to take a closer look at what scipy does to render API docs.
It seems to be rendering as I intended. Look good to you? |
@mdhaber Yes. Looks perfect to me; sorry for the noise. I was under the impression that rst and markdown behave differently here. Clearly, I am wrong and there is some feature/behavior that I am not aware of :) |
Thanks @FirefoxMetzger! |
Closes: #14683
Does two things:
Notes
being placed beforeAttributes