-
Notifications
You must be signed in to change notification settings - Fork 121
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] Added All Regressors (2nd Attempt) #1663
Conversation
Thank you for contributing to
|
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.
LGTM. Thanks for the additional fixes.
@all-contributors add @Moonzyyy for docs |
We had trouble processing your request. Please try again later. |
Uh oh 🙂 |
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.
lgtm
@all-contributors add @Moonzyyy for docs |
We had trouble processing your request. Please try again later. |
@allcontributors add @Moonzyyy for docs |
I've put up a pull request to add @Moonzyyy! 🎉 |
What does this implement/fix? Explain your changes.
Fixes #1500, added all the new regressors to the document and changed the page to have all the regressors in alphabetical order for easier readability. Also fixed a spelling mistake in the installation page.
Also added to the CSS file to fix issues arising with Sphinx .autosummary. An issue would happen where the class.rst template would sometimes push the classh definition to the right of the page. This happened due to margin errors of the class.rst template (at least thats what I figured after messing around with it). This addition to the CSS file has not caused any errors elsewhere that contains .autosummary from what I have checked.
Any other comments?
Had an error with previous merge #1660, apologies.