Simplify parameter logic

This build on !36 (merged) so should be merged after (if it is to be merged)

The Sampler and fill_prior access sampling parameters via the waveform_generator attribute of the likelihood. This therefore places a requirement that the likelihood have a waveform_generator.

To remove this requirement, let's require the likelihood to have a parameters instance and access them this way. For our likelihood, this is as simple as setting them from the waveform_generator.

Further, this will make it easier in future to write different likelihood which may not need a waveform generator, but can handle those issues internally.

This is great.

Just to make sure I understand, likelihood.parameters and waveform_generator.parameters are tied together so updating in likelihood updates in waveform generator?

mentioned in issue #32 (closed)

Will merge it later today

@colm.talbot yes exactly.

merged