TuringLang / TuringLang/AbstractMCMC.jl
(Possibly) get rid of transitions altogether 😱
Open
@penelopeysm is already working on this.
Since Dec 31, 2024.
- Dominant language
- Julia
- Stars
- 108
- Forks
- 20
- PR merge metrics
- No merged PRs in 30d
Description
At today's Turing.jl catchup we discussed the possibility of removing transitions altogether. The reason for this is because the sampler state usually contains all the information required to generate the transition, i.e., the transition contains redundant information and adds unnecessary complexity to the implementation. Concretely, this would mean that:
stepwould return only one value, which is the state;bundle_sampleswould probably be modified to take a vector of states rather than a vector of transitions.- The sampler implementation would be responsible for making sure that there are methods to extract a namedtuple of parameters, logp, or any other quantity that needs to be calculated, from a state rather than a transition. (This is pretty much
getparamsfrom #86.) - Plus more stuff needs to be changed, but that will probably become clearer as we work on it.
(FWIW, I'm very much in favour of this.)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.