Showing 1,558 of 1,565 total issues
Unable to import 'iso8601' Open
import iso8601
- Read upRead up
- Exclude checks
Used when pylint has been unable to import a module.
Cyclic import (apel.db.records -> apel.db.records.cloud) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Cyclic import (apel.parsers -> apel.parsers.slurm) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Cyclic import (apel.parsers -> apel.parsers.htcondor) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Similar lines in 2 files Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Indicates that a set of similar lines has been detected among multiple file. This usually means that the code should be refactored to avoid this duplication. ==apel.db.records.cloud:73 ==apel.db.records.job:93 role, group, vo = parsefqan(self.record_content['FQAN']) # We can't / don't put NULL in the database, so we use 'None' if role is None: role = 'None' if group is None: group = 'None' if vo is None: vo = 'None'
Variable name d
doesn't conform to snake_case naming style Open
d = {}
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Method could be a function Open
def _load_multipliers(self):
- Read upRead up
- Exclude checks
Used when a method doesn't use its bound instance, and so could be written as a function.
standard import import subprocess
should be placed before from apel.db.records.event import EventRecord
Open
import subprocess
- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
standard import import logging
should be placed before from apel.common import parse_time
Open
import logging
- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Cyclic import (apel.db.records -> apel.db.records.group_attribute) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Cyclic import (apel.db.records -> apel.db.records.processed) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Constant name log
doesn't conform to UPPER_CASE naming style Open
log = logging.getLogger(__name__)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Method could be a function Open
def _normalise_memory(self, mem):
- Read upRead up
- Exclude checks
Used when a method doesn't use its bound instance, and so could be written as a function.
Import from .blah import BlahParser
should be placed at the top of the module Open
from .blah import BlahParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed
Variable name fp
doesn't conform to snake_case naming style Open
fp = gzip.open(fname, 'rb')
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Cyclic import (apel.parsers -> apel.parsers.blah) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Similar lines in 2 files Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Indicates that a set of similar lines has been detected among multiple file. This usually means that the code should be refactored to avoid this duplication. ==apel.db.records.normalisedsummary:70 ==apel.db.records.summary:64 def _checkfields(self): ''' Add extra checks to the ones in the parent class. '''
# Call the parent's checks first. Record.checkfields(self)
# shorthand rc = self.recordcontent
monthstart = None monthend = None # For storing the month and year for the subsequent month. nextmonthyear = None next_month = None
try: # A bit convoluted for finding the first second in the next month. if (int(rc['Month']) == 12): nextmonthyear = int(rc['Year']) + 1 nextmonth = 1 else: nextmonthyear = int(rc['Year']) nextmonth = int(rc['Month']) + 1
monthstart = datetime(int(rc['Year']), int(rc['Month']), 1) monthend = datetime(nextmonthyear, next_month, 1)
Line too long (109/100) Open
"userFQAN=/atlas/Role=production/Capability=NULL" "ceID=cream-2-fzk.gridka.de:8443/cream-pbs-atlasXL"
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Variable name rc
doesn't conform to snake_case naming style Open
rc = {}
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Line too long (105/100) Open
'WallDuration' : lambda x: int(round(float(x[13]))*self._get_wall_multiplier(x[1])),
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.