Showing 1,557 of 1,564 total issues
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.
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.normalised_summary:191 ==apel.db.records.summary:186 ur.appendChild(infra)
earliest = doc.createElement('aur:EarliestEndTime') earliesttext = time.strftime('%Y-%m-%dT%H:%M:%SZ', self.getfield('EarliestEndTime').timetuple()) earliest.appendChild(doc.createTextNode(earliest_text)) ur.appendChild(earliest)
latest = doc.createElement('aur:LatestEndTime') latesttext = time.strftime('%Y-%m-%dT%H:%M:%SZ', self.getfield('LatestEndTime').timetuple()) latest.appendChild(doc.createTextNode(latest_text)) ur.appendChild(latest)
wall = doc.createElement('aur:WallDuration') wall.appendChild(doc.createTextNode('PT'+str(self.get_field('WallDuration'))+'S')) ur.appendChild(wall)
cpu = doc.createElement('aur:CpuDuration') cpu.appendChild(doc.createTextNode('PT'+str(self.get_field('CpuDuration'))+'S')) ur.appendChild(cpu)
Wrong continued indentation (add 1 space). Open
'StopTime' : lambda x: int(round(float(x[10])/divisor)),
- Read upRead up
- Exclude checks
TODO 'StopTime' : lambda x: int(round(float(x[10])/divisor)), ^|
Wrong continued indentation (add 1 space). Open
'MemoryReal' : lambda x: int(float(x[37])*1024*1024), # is this correct?
- Read upRead up
- Exclude checks
TODO 'MemoryReal' : lambda x: int(float(x[37])10241024), # is this correct? ^|
Unable to import 'future.builtins' Open
from future.builtins import zip
- Read upRead up
- Exclude checks
Used when pylint has been unable to import a module.
Too many local variables (17/15) Open
def parse(self, line):
- Read upRead up
- Exclude checks
Used when a function or method has too many local variables.
standard import import logging
should be placed before from future.builtins import zip
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)
Variable name ch
doesn't conform to snake_case naming style Open
ch = logging.StreamHandler(sys.stdout)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
standard import import re
should be placed before from apel.db.records.blahd import BlahdRecord
Open
import re
- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Anomalous backslash in string: '.'. String constant might be missing an r prefix. Open
'''
- Read upRead up
- Exclude checks
Used when a backslash is in a literal string but not as an escape.
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...).
Import from .slurm import SlurmParser
should be placed at the top of the module Open
from .slurm import SlurmParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed
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 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...).
standard import import logging
should be placed before from apel.db.records.event import EventRecord
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)
Wrong continued indentation (add 1 space). Open
for x in self.EXPR.findall(line)]
- Read upRead up
- Exclude checks
TODO for x in self.EXPR.findall(line)] ^|
Cyclic import (apel.parsers -> apel.parsers.sge) 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.lsf) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Unnecessary else
after return
Open
if namespace is None:
- Read upRead up
- Exclude checks
Used in order to highlight an unnecessary block of code following an if containing a return statement. As such, it will warn when it encounters an else following a chain of ifs, all of them containing a return statement.
Variable name p
doesn't conform to snake_case naming style Open
p = subprocess.Popen(["qhost", "-F", "-xml"], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).