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

Skip to content

MNT remove unsed _half_multinomial_loss - #34471

Merged
virchan merged 1 commit into
scikit-learn:mainfrom
OmarManzoor:cleanup_array_api
Jul 14, 2026
Merged

MNT remove unsed _half_multinomial_loss#34471
virchan merged 1 commit into
scikit-learn:mainfrom
OmarManzoor:cleanup_array_api

Conversation

@OmarManzoor

Copy link
Copy Markdown
Contributor

Reference Issues/PRs

None

What does this implement/fix? Explain your changes.

  • Remove the _half_multinomial_loss from the array API utils and it's corresponding test
  • This is not used now.

AI usage disclosure

I used AI assistance for:

  • None

Any other comments?

CC: @ogrisel @virchan @lucyleeow

@ogrisel ogrisel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lgtm if the testss are green

@virchan virchan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thanks, @OmarManzoor!

@virchan
virchan merged commit a38b764 into scikit-learn:main Jul 14, 2026
74 of 78 checks passed
@OmarManzoor
OmarManzoor deleted the cleanup_array_api branch July 14, 2026 05:31
prady0t pushed a commit to prady0t/scikit-learn that referenced this pull request Sep 2, 2026
@jeremiedbb jeremiedbb mentioned this pull request Sep 8, 2026
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants