Skip to content

Convert DebugContext to accumulator #974

@penelopeysm

Description

@penelopeysm

Don't think there's any real reason why it needs to be a parent context. The only meaningful things that it could wrap were LikelihoodContext and PriorContext and both of those will be gone, so I think Debug should itself be a leaf.

It would tie in with a general philosophy that:

  • leaf contexts indicate different modes of model execution (i.e., being called by different functions)
    • it is nonsensical to stack multiple leaf contexts
  • parent contexts indicate modifiers to the model itself (i.e., Condition/FixedContext change whether we go down assume or observe, PrefixContext changes the varnames themselves.)
    • multiple parent contexts simply mean multiple modifiers (which in general are not commutative, hence order matters)

(And parent contexts that were commutative have I think mostly been converted to accumulators.)

Metadata

Metadata

Assignees

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