Skip to content

DOC Add link to random tree embedding example in docs#30418

Merged
adrinjalali merged 3 commits intoscikit-learn:mainfrom
hanjunkim11:add-RandomTreesEmbedding-example
Jan 2, 2025
Merged

DOC Add link to random tree embedding example in docs#30418
adrinjalali merged 3 commits intoscikit-learn:mainfrom
hanjunkim11:add-RandomTreesEmbedding-example

Conversation

@hanjunkim11
Copy link
Copy Markdown
Contributor

Reference Issues/PRs
#26927

What does this implement/fix? Explain your changes.
Added random forest embedding example reference to the docstring of RandomTreesEmbedding class in _forest.py.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 6, 2024

✔️ Linting Passed

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

Generated for commit: 1d05999. Link to the linter CI: here

Copy link
Copy Markdown
Member

@virchan virchan left a comment

Choose a reason for hiding this comment

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

Thank you for the PR @hanjunkim11! I have a suggestion.

Copy link
Copy Markdown
Member

@virchan virchan left a comment

Choose a reason for hiding this comment

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

LGTM!

@adrinjalali, could you please take a look and merge this?

@adrinjalali adrinjalali merged commit 7a98fd2 into scikit-learn:main Jan 2, 2025
@hanjunkim11 hanjunkim11 deleted the add-RandomTreesEmbedding-example branch January 2, 2025 17:44
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Jan 8, 2025
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