reduced errors related to styleguides

pycopancore/model_components/seven_dwarfs/implementation/individual.py

@@ -19,7 +19,7 @@
 import numpy as np
 
 
-class Individual (I.Individual):
+class Individual(I.Individual):
  """Individual entity type mixin implementation class."""
 
  # standard methods:
@@ -79,13 +79,12 @@ def beard_growing(self, t):
  )
 
  def reproduction(self, unused_t):
- """Reproduce. 
- 
+ """Reproduce.
+
  Done if dwarf is older than 5, not dead and has enough to eat.
  """
- if self.is_active \
- and self.cell.eating_stock > 10\
- and self.age > 5:
+ if (self.is_active
+ and self.cell.eating_stock > 10 and self.age > 5):
  child = self.__class__(cell=self.cell,
  age=1,
  beard_length=0,

pycopancore/model_components/seven_dwarfs/model.py

@@ -18,7 +18,7 @@
 # import all needed process taxon implementation classes:
 
 
-class Model (I.Model):
+class Model(I.Model):
  """Model mixin class."""
 
  # mixins provided by this model component:

pycopancore/model_components/simple_extraction/implementation/cell.py

@@ -29,19 +29,17 @@ def __init__(self,
  self.stock = stock
  self.growth_rate = growth_rate
 
- pass
 
  def deactivate(self):
  """Deactivate a cell."""
  # TODO: add custom code here:
- pass
+
  super().deactivate() # must be the last line
 
  def reactivate(self):
  """Reactivate a cell."""
  super().reactivate() # must be the first line
  # TODO: add custom code here:
- pass
 
  # process-related methods:
 

pycopancore/model_components/simple_extraction/implementation/individual.py

@@ -25,7 +25,6 @@ def __init__(self,
  super().__init__(**kwargs) # must be the first line
  self.strategy = strategy
 
- pass
 
  def __lt__(self, other):
  """Make objects sortable."""
@@ -34,14 +33,14 @@ def __lt__(self, other):
  def deactivate(self):
  """Deactivate an individual."""
  # TODO: add custom code here:
- pass
+
  super().deactivate() # must be the last line
 
  def reactivate(self):
  """Reactivate an individual."""
  super().reactivate() # must be the first line
  # TODO: add custom code here:
- pass
+
 
  # process-related methods:
 

pycopancore/model_components/simple_extraction/model.py

@@ -15,7 +15,7 @@
 # none needed
 
 
-class Model (I.Model):
+class Model(I.Model):
  """Model mixin class."""
 
  # mixins provided by this model component:

pycopancore/private/_abstract_process_taxon_mixin.py

@@ -38,15 +38,17 @@ def __init__(self):
  # the repr and the str methods were removed in the master/prototype_jobst1
  # Do we really don't want them anymore?
  def __repr__(self):
- return ('Process taxon object')
+ return 'Process taxon object'
 
  def __str__(self):
  return repr(self)
 
- def set_value(self, variable, value):
- assert isinstance(variable, variable.Variable), \
+ def set_value(self, var, value):
+ """Dummy docstring"""
+ # TODO: missing method docstring
+ assert isinstance(var, variable.Variable), \
  "variable must be a Variable object"
- variable.set_value(self, value)
+ var.set_value(self, value)
 
  def assert_valid(self):
  """Make sure all variable values are valid.

pycopancore/process_types/explicit.py

@@ -35,7 +35,7 @@ def __init__(self,
  targets,
  specification,
  smoothness=0
- ):
+  ):
  """Instantiate an instance of an explicit process.
 
  Parameters

pycopancore/runners/hooks.py

@@ -18,10 +18,14 @@
 
 
 class HookRegistrationError(BaseException):
+ """Dummy docstring"""
+ # TODO: missing class docstring
  pass
 
 
 class HooksError(BaseException):
+ """Dummy docstring"""
+ # TODO: missing class docstring
  pass
 
 #
@@ -143,6 +147,8 @@ def unregister_hook(cls, type, hook, theclass=None,
 
  @classmethod
  def execute_hooks(cls, type, model, t):
+ """Dummy docstring"""
+ # TODO: Missing method docstring
  assert type in cls.Types, "please give a type from {}.HookTypes".format(
  cls.__qualname__)
  if type is cls.Types.pre:

pytest.ini

@@ -8,7 +8,7 @@ norecursedirs =
 format = parsable
 skip = */__init__.py,setup.py,*/conf.py
 linters = pylint,mccabe,pep257,pep8,pyflakes,pycodestyle,pydocstyle
-ignore = D203,D213,E741,W0212,C0411,W0622,E501,E712,W0640,C0301,C0121,R0204,W0511
+ignore = D203,D213,E741,W0212,C0411,W0622,E501,E712,W0640,C0301,C0121,R0204,W0511,C0112,C0330,W0702,C0326
 # W0511 to ignore TODO statements