Should base-clocks/models/priors be abstract?

[Anaphory]

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 #248