WIP MCMC with transforms Interface
FYI @oschulz @Cornelius-G
Can you check the
Base.Meta.ParseError("parsing error in src/samplers/transformed_mcmc/struct_list.jl:34: missing comma or } in argument list")
CI error on Julia 1?
Codecov Report
Patch coverage has no change and project coverage change: -5.76% :warning:
Comparison is base (
78e9339) 55.06% compared to head (9cb2625) 49.31%.
:exclamation: Current head 9cb2625 differs from pull request most recent head b6565c3. Consider uploading reports for the commit b6565c3 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## main #409 +/- ##
==========================================
- Coverage 55.06% 49.31% -5.76%
==========================================
Files 116 131 +15
Lines 5619 6323 +704
==========================================
+ Hits 3094 3118 +24
- Misses 2525 3205 +680
| Files Changed | Coverage Δ | |
|---|---|---|
| src/BAT.jl | 100.00% <ø> (ø) |
|
| src/samplers/transformed_mcmc/chain_pool_init.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_algorithm.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_convergence.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_iterate.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_sample.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_sampleid.jl | 0.00% <0.00%> (ø) |
|
| src/samplers/transformed_mcmc/mcmc_stats.jl | 0.00% <0.00%> (ø) |
|
| ...rs/transformed_mcmc/mcmc_tuning/mcmc_noop_tuner.jl | 0.00% <0.00%> (ø) |
|
| ...sformed_mcmc/mcmc_tuning/mcmc_proposalcov_tuner.jl | 0.00% <0.00%> (ø) |
|
| ... and 7 more |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I've updated this to use the new BATContext. "transformed_example.jl" doesn't fully run through, but I'm not sure if that's related.
TransformedAdaptiveMHTuning in the example works now, but
r_hmc = @time BAT.bat_sample_impl(posterior, MCMCSampling(mcalg=HamiltonianMC(), nchains=4, nsteps=4*20000), context)
in the example doesn't work yet, but that's to be expected currently, right?
I think BATContext-wise this is fine now.
TransformedAdaptiveMHTuningin the example works now, butr_hmc = @time BAT.bat_sample_impl(posterior, MCMCSampling(mcalg=HamiltonianMC(), nchains=4, nsteps=4*20000), context)in the example doesn't work yet, but that's to be expected currently, right?
Hm, for me this line works. As it should, because that is just the "good old" HMC smapling, not using the new transformations.
Ah - well, in any case, I think the adaption to BATContext is complete.
@Micki-D needs to be updated to API changes.
Closing in favor of #441