VerifyCedarCluster.get_arguments() - Code Metrics - Inspection of "Add configuration file for Cedar" - SMART-Lab/smartdispatch - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#173)

unknown

created 2017-10-17 20:24 UTC

VerifyCedarCluster.get_arguments() A

↳ Parent: VerifyCedarCluster

Complexity

Conditions

Size

Total Lines

Duplication

Lines	15
Ratio	100 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	4
c	2
b	0
f	0
dl	15
loc	15
rs	9.2

import os
import sys

from verify_slurm_cluster import VerifySlurmCluster, set_defaults


class VerifyCedarCluster(VerifySlurmCluster):


    WALLTIME = 60
    CORES_PER_NODE = 24
    GPUS_PER_NODE = 4

    def get_arguments(self, **kwargs):

        kwargs = super(VerifyCedarCluster, self).get_arguments(**kwargs)

        if kwargs["gpusPerCommand"] == 0:
            account = os.environ.get("CPU_SLURM_ACCOUNT")
        else:
            account = os.environ.get("GPU_SLURM_ACCOUNT")

        if "sbatchFlags" not in kwargs or len(kwargs["sbatchFlags"]) == 0:
            kwargs["sbatchFlags"] = "--account=" + account
        else:
            kwargs["sbatchFlags"] += " --account=" + account

        return kwargs


if __name__ == "__main__":
    verifications = filter(lambda o: not o.startswith("--"), sys.argv[1:])
    VerifyCedarCluster(debug="--debug" in sys.argv[1:],
                       no_fork="--no-fork" in sys.argv[1:]).run_verifications(
        filtered_by=verifications)


1		import os
2		import sys
3
4		from verify_slurm_cluster import VerifySlurmCluster, set_defaults
5
6
7	View Code Duplication	class VerifyCedarCluster(VerifySlurmCluster):
		0 ignored issues – show Duplication introduced 2017-10-17 20:23 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
8
9		WALLTIME = 60
10		CORES_PER_NODE = 24
11		GPUS_PER_NODE = 4
12
13		def get_arguments(self, **kwargs):
14
15		kwargs = super(VerifyCedarCluster, self).get_arguments(**kwargs)
16
17		if kwargs["gpusPerCommand"] == 0:
18		account = os.environ.get("CPU_SLURM_ACCOUNT")
19		else:
20		account = os.environ.get("GPU_SLURM_ACCOUNT")
21
22		if "sbatchFlags" not in kwargs or len(kwargs["sbatchFlags"]) == 0:
23		kwargs["sbatchFlags"] = "--account=" + account
24		else:
25		kwargs["sbatchFlags"] += " --account=" + account
26
27		return kwargs
28
29
30		if __name__ == "__main__":
31		verifications = filter(lambda o: not o.startswith("--"), sys.argv[1:])
32		VerifyCedarCluster(debug="--debug" in sys.argv[1:],
33		no_fork="--no-fork" in sys.argv[1:]).run_verifications(
34		filtered_by=verifications)
35

SMART-Lab / smartdispatch

Pull Request — master (#173)

VerifyCedarCluster.get_arguments() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like