Skip to content
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

fix flake8 errors due to new pypi dependency/rule updates #597

Merged

Conversation

imatiach-msft
Copy link
Collaborator

@imatiach-msft imatiach-msft commented May 30, 2024

Fix new flake8 errors that started to appear in gated builds after new versions of flake8 helper packages were released with new rules.
Specifically, I believe these are from flake8-bugbear and flake8-pytest-style updates.
Also temporarily pin scikit-learn<=1.4.2 due to issue with econml and scikit-learn in tests when validating ExplanationDashboard visualization from raiwidgets package.

Copilot summary:

This pull request primarily involves syntax modifications across several Python test files. The most significant changes include the addition of a specific error code to a BaseException in gpu_kmeans.py, and the removal of parentheses from several pytest decorators in conftest.py and test_notebooks.py.

Here are the key changes:

Syntax modifications:

@imatiach-msft imatiach-msft merged commit d11712b into interpretml:main May 31, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants