Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MAINT Sync hdbscan branch with main #25124

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

Closed
wants to merge 260 commits into from

Conversation

Micky774
Copy link
Contributor

@Micky774 Micky774 commented Dec 6, 2022

Reference Issues/PRs

Towards #24686

What does this implement/fix? Explain your changes.

Syncs with main and moves setup/build configuration to setup.py.

Any other comments?

jeremiedbb and others added 30 commits October 13, 2022 14:54
…24653)

* remove load_boston references from the docs

* iter

* update poor_score
…CV (scikit-learn#24543)


Co-authored-by: Tim Head <[email protected]>
Co-authored-by: Olivier Grisel <[email protected]>
Co-authored-by: Loïc Estève <[email protected]>
* DOC Improve docstring around set_output

* DOC Improve docs around set_output

* DOC Address comments

* DOC Better grammar

* DOC Improve wording

* DOC Improves docstring in set_config
Co-authored-by: Julien Jerphanion <[email protected]>
Co-authored-by: Thomas J. Fan <[email protected]>
gu1show and others added 22 commits December 1, 2022 07:57
Co-authored-by: Tim Head <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
Co-authored-by: Guillaume Lemaitre <[email protected]>
…n#25083)

* Doc changed n_init to n_jobs in mean_shift.py
@Micky774 Micky774 added the Quick Review For PRs that are quick to review label Dec 6, 2022
@github-actions github-actions bot added the cython label Dec 6, 2022
@thomasjpfan
Copy link
Member

I think sklearn/cluster/_hdbscan/setup.py still needs to be removed. For the future, with big syncs like this, it's nice to have the change you made be in it's own commit so it's easier to review. In this case, it's the migration to the root setup.py.

I checked locally that squashing and merging this into hdbscan does not make it too hard to merge the feature branch, hdbscan, into main.

@Micky774 Micky774 closed this Dec 7, 2022
@Micky774 Micky774 deleted the sync_hdbscan branch December 7, 2022 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cython No Changelog Needed Quick Review For PRs that are quick to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.