Coding style fixes

Author: Vasileios Karakasis

cscs-checks/apps/vasp/vasp_check.py

@@ -11,19 +11,17 @@ class VASPCheck(rfm.RunOnlyRegressionTest):
     def __init__(self):
         self.valid_prog_environs = ['builtin']
         self.modules = ['VASP']
-
         force = sn.extractsingle(r'1 F=\s+(?P<result>\S+)',
                                  self.stdout, 'result', float)
         self.sanity_patterns = sn.assert_reference(
-            force, -.85026214E+03, -1e-5, 1e-5)
+            force, -.85026214E+03, -1e-5, 1e-5
+        )
         self.keep_files = ['OUTCAR']
-
         self.perf_patterns = {
             'time': sn.extractsingle(r'Total CPU time used \(sec\):'
                                      r'\s+(?P<time>\S+)', 'OUTCAR',
                                      'time', float)
         }
-
         self.maintainers = ['LM']
         self.tags = {'scs'}
         self.strict_check = False
@@ -34,12 +32,11 @@ def __init__(self):
         }
 
 
-@rfm.parameterized_test(*([v]
-                          for v in ['maint', 'prod']))
+@rfm.parameterized_test(*([v] for v in ['maint', 'prod']))
 class VASPCpuCheck(VASPCheck):
     def __init__(self, variant):
         super().__init__()
-        self.descr = 'VASP CPU check (version: %s)' % variant
+        self.descr = f'VASP CPU check (variant: {variant})'
         self.valid_systems = ['daint:mc', 'dom:mc', 'eiger:mc']
         self.executable = 'vasp_std'
         if self.current_system.name == 'dom':
@@ -75,7 +72,6 @@ def __init__(self, variant):
                 'eiger:mc': {'time': (100.0, None, 0.10, 's')}
             }
         }
-
         self.reference = references[variant]
         self.tags |= {'maintenance' if variant == 'maint' else 'production'}
 
@@ -88,16 +84,14 @@ def set_cpu_binding(self):
         self.job.launcher.options = ['--cpu-bind=cores']
 
 
-@rfm.parameterized_test(*([v]
-                          for v in ['maint', 'prod']))
+@rfm.parameterized_test(*([v] for v in ['maint', 'prod']))
 class VASPGpuCheck(VASPCheck):
     def __init__(self, variant):
         super().__init__()
-        self.descr = 'VASP GPU check (version: %s)' % variant
+        self.descr = f'VASP GPU check (variant: {variant})'
         self.valid_systems = ['daint:gpu', 'dom:gpu']
         self.executable = 'vasp_gpu'
         self.variables = {'CRAY_CUDA_MPS': '1'}
-
         self.num_gpus_per_node = 1
         if self.current_system.name == 'dom':
             self.num_tasks = 6
@@ -116,6 +110,5 @@ def __init__(self, variant):
                 'daint:gpu': {'time': (46.7, None, 0.20, 's')},
             }
         }
-
         self.reference = references[variant]
         self.tags |= {'maintenance' if variant == 'maint' else 'production'}