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

BUG: fix newline #12117

Merged
merged 2 commits into from May 15, 2020
Merged

BUG: fix newline #12117

merged 2 commits into from May 15, 2020

Conversation

flxai
Copy link
Contributor

@flxai flxai commented May 14, 2020

Reference issue

No issue exists.

What does this implement/fix?

Adds a missing newline in scipy/optimize/_differentialevolution.py so the examples' code blocks render correctly.

@miladsade96 miladsade96 added scipy.optimize Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org labels May 14, 2020
Copy link
Member

@ilayn ilayn left a comment

Choose a reason for hiding this comment

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

@flxai Thanks for this PR. While we are at it, do you mind touching also line 198 to use straight quotes instead of backticks to render best and best1bin as normal text (as in line 208-209). Single backticks are used for internal referencing so that's why it is not rendering correctly.

@flxai
Copy link
Contributor Author

flxai commented May 15, 2020

@ilayn Thanks for the suggestion.

@hameerabbasi
Copy link
Contributor

hameerabbasi commented May 15, 2020

The issue in the CI is a Numba error: Numba version 0.49 requires NumPy 1.15 minimum.

@flxai
Copy link
Contributor Author

flxai commented May 15, 2020

@hameerabbasi Does that mean it can be safely ignored?

@hameerabbasi
Copy link
Contributor

Yes, it's unrelated to this PR. I will try to make a PR to fix it.

@ilayn ilayn merged commit ec16ddc into scipy:master May 15, 2020
@ilayn
Copy link
Member

ilayn commented May 15, 2020

Thanks @flxai, @hameerabbasi

@tylerjereddy tylerjereddy added this to the 1.5.0 milestone May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.optimize
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants