Skip to content

del "del": remove del flag #982

@penelopeysm

Description

@penelopeysm

Once InitContext is fully merged, SampleFrom{Prior,Uniform}, as well as setval_and_resample!, will have been removed. This means that the only code left that uses the del flag is upstream in Turing's pMCMC samplers.

Following TuringLang/Turing.jl#2629, it is clear that there is no need for a per-varname del flag on VarInfo. A single VarInfo-wide boolean, del flag would suffice. Thus at the very least this simplification should be implemented.

Going beyond that, we could potentially manage this using contexts instead of a del flag. That will require some extra investigation.

Note that this post has been edited based on upstream changes in Turing.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions