BaseClock doesn't have a branch rate model, which to me makes that method pretty abstract, so the whole class is? It also lacks __type__ and __distribution__, and declaring it abstract would prevent an accidental instantiation if after some refactoring [clock] type=base would theoretically run etc.
BaseClock doesn't have a branch rate model, which to me makes that method pretty abstract, so the whole class is? It also lacks
__type__
and__distribution__
, and declaring it abstract would prevent an accidental instantiation if after some refactoring[clock] type=base
would theoretically run etc.Originally posted by @Anaphory in https://github.com/lmaurits/BEASTling/pull/248