Skip to content

Commit

Permalink
Remove the prob and logprob macros (#604)
Browse files Browse the repository at this point in the history
* remove files

* add alternative prob macro impl back to warn user

* export again

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* remove the macros

* Update src/DynamicPPL.jl

Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com>

* Update src/DynamicPPL.jl

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com>
  • Loading branch information
3 people committed May 15, 2024
1 parent 48487cc commit 0c76bf2
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 423 deletions.
4 changes: 0 additions & 4 deletions src/DynamicPPL.jl
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,6 @@ export AbstractVarInfo,
# Pseudo distributions
NamedDist,
NoDist,
# Prob macros
@prob_str,
@logprob_str,
# Convenience functions
logprior,
logjoint,
Expand Down Expand Up @@ -172,7 +169,6 @@ include("varinfo.jl")
include("simple_varinfo.jl")
include("context_implementations.jl")
include("compiler.jl")
include("prob_macro.jl")
include("loglikelihoods.jl")
include("submodel_macro.jl")
include("test_utils.jl")
Expand Down
244 changes: 0 additions & 244 deletions src/prob_macro.jl

This file was deleted.

72 changes: 0 additions & 72 deletions test/prob_macro.jl

This file was deleted.

1 change: 0 additions & 1 deletion test/runtests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ include("test_util.jl")
include("simple_varinfo.jl")
include("model.jl")
include("sampler.jl")
include("prob_macro.jl")
include("independence.jl")
include("distribution_wrappers.jl")
include("contexts.jl")
Expand Down
Loading

0 comments on commit 0c76bf2

Please sign in to comment.