Skip to content

Replaced unsigned char with uint8_t #29596

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Replaced unsigned char with uint8_t #29596

wants to merge 1 commit into from

Conversation

norgera
Copy link
Contributor

@norgera norgera commented Jul 31, 2024 •
edited
Loading

No description provided.

@github-actions GitHub Actions
Copy link

❌ Linting issues

This PR is introducing linting issues. Here's a summary of the issues. Note that you can avoid having linting issues by enabling pre-commit hooks. Instructions to enable them can be found here.

You can see the details of the linting issues under the lint job here

cython-lint

cython-lint detected issues. Please fix them locally and push the changes. Here you can see the detected issues. Note that the installed cython-lint version is cython-lint=0.16.2.


/home/runner/work/scikit-learn/scikit-learn/sklearn/ensemble/_hist_gradient_boosting/_bitset.pyx:29:37: E127 continuation line over-indented for visual indent
/home/runner/work/scikit-learn/scikit-learn/sklearn/ensemble/_hist_gradient_boosting/_bitset.pyx:35:49: E127 continuation line over-indented for visual indent
/home/runner/work/scikit-learn/scikit-learn/sklearn/ensemble/_hist_gradient_boosting/_bitset.pyx:39:51: E127 continuation line over-indented for visual indent
/home/runner/work/scikit-learn/scikit-learn/sklearn/ensemble/_hist_gradient_boosting/_bitset.pyx:40:51: E127 continuation line over-indented for visual indent
/home/runner/work/scikit-learn/scikit-learn/sklearn/ensemble/_hist_gradient_boosting/_bitset.pxd:13:42: E127 continuation line over-indented for visual indent

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

@norgera norgera closed this Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cython module:ensemble module:linear_model module:metrics
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@norgera