Skip to content

DOC Adding dropdown for 1.17. Neural network models (supervised)#27612

Closed
KartikeyBartwal wants to merge 2 commits intoscikit-learn:mainfrom
KartikeyBartwal:neural_networks_supervised
Closed

DOC Adding dropdown for 1.17. Neural network models (supervised)#27612
KartikeyBartwal wants to merge 2 commits intoscikit-learn:mainfrom
KartikeyBartwal:neural_networks_supervised

Conversation

@KartikeyBartwal
Copy link
Copy Markdown

Reference Issues/PRs

Adding dropdowns for section 1.17. Neural network models (supervised) mentioned in #26617

What does this implement/fix? Explain your changes.

Added dropdowns for the following:

  1. Figure 1: One hidden layer MLP and its description
  2. Details about 'coefs_' and 'intercepts_'
  3. Advantages of Multi-layer Perceptron
  4. Disadvantages of Multi-layer Perceptron
  5. Tips on Practical Use
  6. References

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: e3ba0ae. Link to the linter CI: here

@ArturoAmorQ
Copy link
Copy Markdown
Member

Thanks for the PR @KartikeyBartwal, and sorry if the instructions on how to add dropdown where not clear enough.

See Contributing to documentation, notably the "Guidelines for writing the User Guide and other reStructuredText documents" dropdown, where we explain the syntax

|details-start|
**Dropdown title**
|details-split|

Dropdown content.

|details-end|

@ArturoAmorQ
Copy link
Copy Markdown
Member

Hi @KartikeyBartwal, are you still working on this PR?
Please let us know if you need help on how to proceed.

@KartikeyBartwal
Copy link
Copy Markdown
Author

Hi @KartikeyBartwal, are you still working on this PR? Please let us know if you need help on how to proceed.

Yes, I will be completing this task. Kindly bear with me

@ArturoAmorQ
Copy link
Copy Markdown
Member

I am sorry, @KartikeyBartwal, I am closing this PR as a duplicate of #27920. Please open a new PR if you want to iterate over the work introduced there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants