Showing 1,558 of 1,565 total issues

Variable name sl doesn't conform to snake_case naming style
Open

        (slt, sl) = self._check_factor(slt, sl)
Severity: Info
Found in apel/db/records/job.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Unnecessary else after raise
Open

            if sfu == 'None':
Severity: Info
Found in apel/db/records/job.py by pylint

Used in order to highlight an unnecessary block of code following an if containing a raise statement. As such, it will warn when it encounters an else following a chain of ifs, all of them containing a raise statement.

standard import from xml.dom.minidom import Document should be placed before from future.builtins import str
Open

from xml.dom.minidom import Document
Severity: Info
Found in apel/db/records/job.py by pylint

Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)

Line too long (101/100)
Open

        nwall.appendChild(doc.createTextNode('PT'+str(self.get_field('NormalisedWallDuration'))+'S'))

Used when a line is longer than a given number of characters.

Too many local variables (23/15)
Open

    def get_ur(self):

Used when a function or method has too many local variables.

standard import import logging should be placed before from future.builtins import object, str, zip
Open

import logging
Severity: Info
Found in apel/db/records/record.py by pylint

Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)

standard import import time should be placed before from future.builtins import str
Open

import time
Severity: Info
Found in apel/db/records/storage.py by pylint

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 future.builtins import str
Open

import logging
Severity: Info
Found in apel/db/records/storage.py by pylint

Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)

Variable name UR_OPEN doesn't conform to snake_case naming style
Open

            UR_OPEN = ('<sr:StorageUsageRecords xmlns:sr="http://eu-emi.eu/namespaces/2011/02/storagerecord">')
Severity: Info
Found in apel/db/unloader.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Wrong continued indentation (add 14 spaces).
Open

                    "CpuDuration", "NumberOfJobs"]
Severity: Info
Found in apel/db/records/summary.py by pylint

TODO CpuDuration, NumberOfJobs] ^ |

Argument name sf doesn't conform to snake_case naming style
Open

    def _check_factor(self, sfu, sf):
Severity: Info
Found in apel/db/records/job.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Line too long (101/100)
Open

                    # A trailing Z in the ISO format denotes UTC.  We make this explicit for parsing.
Severity: Info
Found in apel/db/records/record.py by pylint

Used when a line is longer than a given number of characters.

Constant name log doesn't conform to UPPER_CASE naming style
Open

log = logging.getLogger(LOGGER_ID)
Severity: Info
Found in apel/db/records/record.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Too many instance attributes (9/7)
Open

class Record(object):
Severity: Info
Found in apel/db/records/record.py by pylint

Used when class has too many instance attributes, try to reduce this to get a simpler (and so easier to use) class.

Variable name dn doesn't conform to snake_case naming style
Open

        dn = self.get_field(Record.DN_FIELD)
Severity: Info
Found in apel/db/records/record.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Wrong hanging indentation (remove 6 spaces).
Open

              EventRecord : 'CALL ReplaceEventRecord(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)',
Severity: Info
Found in apel/db/backends/mysql.py by pylint

TODO EventRecord : 'CALL ReplaceEventRecord(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)', | ^

Variable name XML_HEADER doesn't conform to snake_case naming style
Open

            XML_HEADER = '<?xml version="1.0" ?>'
Severity: Info
Found in apel/db/unloader.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Variable name XML_HEADER doesn't conform to snake_case naming style
Open

            XML_HEADER = '<?xml version="1.0" ?>'
Severity: Info
Found in apel/db/unloader.py by pylint

Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).

Import from apel.db.apeldb import ApelDb, Query, ApelDbException should be placed at the top of the module
Open

from apel.db.apeldb import ApelDb, Query, ApelDbException
Severity: Info
Found in apel/db/__init__.py by pylint

Used when code and imports are mixed

standard import from datetime import datetime, timedelta should be placed before from future.builtins import str
Open

from datetime import datetime, timedelta
Severity: Info
Found in apel/db/records/cloud.py by pylint

Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)

Severity
Category
Status
Source
Language