Fixes #106, allow external samplers.

[zeyus]

Added typing to fit_model and added test.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.05%. Comparing base (bb44028) to head (8fc13ee).

Additional details and impacted files

@@             Coverage Diff             @@
##              dev     #115       +/-   ##
===========================================
- Coverage   78.73%   68.05%   -10.68%     
===========================================
  Files          26       26               
  Lines         743      742        -1     
===========================================
- Hits          585      505       -80     
- Misses        158      237       +79     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[PTWaade]

Is there a shared supertype for AbstractSampler and InferenceAlgorithm we can use..? Otherwise, this looks great

[zeyus]

Is there a shared supertype for AbstractSampler and InferenceAlgorithm we can use..? Otherwise, this looks great

Unfortunately I don't think there is, I had a look through the different function signatures for sample and I could see definitions for both. I checked the sulertypes and neither inherit from a common base...

The only thing I thought of is they need to implement the the step method and one other (but I couldn't find a way to say in Julia that it should fit this kind of interface?) if it's possible that would be better than the union type