Make `BUGSModel` parametric so Tapir works with JuliaBUGS models

sunxd3 commented 3 months ago

Fix https://github.com/TuringLang/JuliaBUGS.jl/issues/202

sunxd3 commented 3 months ago

compat errors on 1.9. @willtebbutt how should I resolve this?

willtebbutt commented 3 months ago

Ugh. Sadly you'll have to do something like this, and only run the Tapir.jl tests if you happen to be testing on version 1.10 or greater.

Sorting this out is on my todo list.

coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 10146783820

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.302%

Totals
Change from base Build 9928535952:	0.0%
Covered Lines:	1437
Relevant Lines:	1746

💛 - Coveralls

sunxd3 commented 3 months ago

I don't think I can do that without causing an error: I can't have Tapir in the dep/extras field, and Julia complains when I try to restrict the pkg version when it's not in dep/extras

willtebbutt commented 3 months ago

Ohhhh ofc, you're completely right 👍

TuringLang / JuliaBUGS.jl