Class docstring is missing. If you want to ignore this, you can configure this in the .deepsource.toml file. Please refer to docs for available options.
Occurrences
There are 10 occurrences of this issue in the repository.
The use of else or elif becomes redundant and can be dropped if the last statement under the leading if / elif block is a return statement. In the case of an elif after return, it can be written as a separate if block. For else blocks after return, the โฆ
Occurrences
There are 2 occurrences of this issue in the repository.
When an if-else-if chain has identical conditions, a duplicate will evaluate the same true or false value as the identical expression earlier in the chain, meaning that its branch can never execute. The issue can be autofixed by DeepSource by removing the duplicate if statement and its code block.
Occurrences
There is 1 occurrence of this issue in the repository.
Defining an instance attribute outside __init__ affects the readability of code. It is expected to find all the attributes an instance may have by reading its __init__ method. If there is a need to initialize attribute via sub-initialization methods, it is recommended to assign attributes to None in the init โฆ
Occurrences
There are 4 occurrences of this issue in the repository.
Defining a local variable or function with the same name as a built-in object makes the built-in object unusable within the current scope and makes the code prone to bugs.
Occurrences
There is 1 occurrence of this issue in the repository.
An unused argument can lead to confusions. It should be removed. If this variable is necessary, name the variable _ or start the name with unused or _unused.
Occurrences
There is 1 occurrence of this issue in the repository.
If the except block catches a very general exception, it is likely to catch any unrelated errors too. Try to be more explicit about which exception(s) you're trying to catch.
Occurrences
There are 7 occurrences of this issue in the repository.
A local name is redefining an argument, which might suggest a potential error. This is taken into account only for a handful of name binding operations, such as for iterations, with statements, and exception handler assignments.
Occurrences
There are 2 occurrences of this issue in the repository.
An unused variable takes up space in the code, and can lead to confusion, and it should be removed. If this variable is necessary, name the variable _ to indicate that it will be unused, or start the name with unused or _unused.
Occurrences
There is 1 occurrence of this issue in the repository.
Python's open() function can take in a relative or absolute path and read its file contents. If a user is provided direct access to the path that is opened, it can have serious security risks.
Occurrences
There is 1 occurrence of this issue in the repository.
The logging statement has the call of the form logging.(format_string % (format_args...)). For such calls, it is recommended to leave string interpolation to the logging method itself and be written as logging.(format_string, format_args...) so that the program may avoid incurring the cost of the interpolation in those cases in which โฆ
Occurrences
There is 1 occurrence of this issue in the repository.