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

Skip to content

FIX accept NumPy arrays for alphas in GraphicalLassoCV - #22493

Merged
thomasjpfan merged 5 commits into
scikit-learn:mainfrom
glemaitre:is/22489
Feb 15, 2022
Merged

FIX accept NumPy arrays for alphas in GraphicalLassoCV#22493
thomasjpfan merged 5 commits into
scikit-learn:mainfrom
glemaitre:is/22489

Conversation

@glemaitre

Copy link
Copy Markdown
Member

closes #22489

I checked and it was the only location where we had this issue with collections.abc.Sequence.

@jeremiedbb jeremiedbb 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 @glemaitre

@jeremiedbb jeremiedbb added the Quick Review For PRs that are quick to review label Feb 15, 2022

@thomasjpfan thomasjpfan 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

@thomasjpfan
thomasjpfan merged commit abbee57 into scikit-learn:main Feb 15, 2022
thomasjpfan pushed a commit to thomasjpfan/scikit-learn that referenced this pull request Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:covariance Quick Review For PRs that are quick to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should not use collections.abc.Sequence to ducktype ndarray

3 participants