Skip to content

FAI-881: Upgrade Pylint version #112

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

Merged
merged 2 commits into from
Nov 21, 2022
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
@@ -32,7 +32,7 @@ dev = [
"joblib==1.0.1",
"jupyter==1.0.0",
"scikit-learn==0.24.2",
"pylint==2.9.0",
"pylint==2.15.6",
"pytest==6.2.4",
"setuptools",
"wheel",
2 changes: 1 addition & 1 deletion requirements-dev.txt
Original file line number Diff line number Diff line change
@@ -4,7 +4,7 @@ click==8.0.4
joblib==1.2.0
jupyter==1.0.0
scikit-learn==0.24.2
pylint==2.9.0
pylint==2.15.6
pytest==6.2.4
setuptools
wheel
1 change: 1 addition & 0 deletions src/trustyai/explainers.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
"""Explainers module"""
# pylint: disable = import-error, too-few-public-methods, wrong-import-order, line-too-long,
# pylint: disable = unused-argument, too-many-lines
# pylint: disable = consider-using-f-string, invalid-name
from typing import Dict, Optional, List, Union
import matplotlib.pyplot as plt
import matplotlib as mpl
1 change: 1 addition & 0 deletions src/trustyai/initializer.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# pylint: disable = import-error, import-outside-toplevel, dangerous-default-value, invalid-name, R0801
# pylint: disable = deprecated-module
"""Main TrustyAI Python bindings"""
from distutils.sysconfig import get_python_lib
import glob
1 change: 1 addition & 0 deletions src/trustyai/model/__init__.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
# pylint: disable = import-error, too-few-public-methods, invalid-name, duplicate-code, too-many-lines
# pylint: disable = unused-import, wrong-import-order
# pylint: disable = consider-using-f-string
"""General model classes"""
import uuid as _uuid
from typing import List, Optional, Union, Callable
2 changes: 1 addition & 1 deletion src/trustyai/utils/_tyrus_info_text.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# pylint: disable = consider-using-f-string
"""Info text used in Tyrus visualization explainer info"""
from trustyai.utils._visualisation import bold_red_html, bold_green_html


LIME_TEXT = """
<div style="padding: 10px">
<h3> What is LIME?</h3>
2 changes: 1 addition & 1 deletion src/trustyai/utils/_visualisation.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
"""Visualiser utilies for explainer results"""
# pylint: disable = consider-using-f-string
from abc import ABC, abstractmethod
import pandas as pd
from pandas.io.formats.style import Styler
@@ -52,7 +53,6 @@ def feature_html(content):
"neutral_primary_colour": "#ffffff",
}


DEFAULT_RC_PARAMS = {
"patch.linewidth": 0.5,
"patch.facecolor": "348ABD",
1 change: 1 addition & 0 deletions src/trustyai/utils/tyrus.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
"""Tyrus module"""
# pylint: disable = too-few-public-methods, wrong-import-order, protected-access, cell-var-from-loop
# pylint: disable = too-many-instance-attributes, import-error. too-many-locals
# pylint: disable = consider-using-f-string
import numpy as np
import pandas as pd
from bokeh.io import show, output_file, output_notebook, reset_output