-
Notifications
You must be signed in to change notification settings - Fork 36
Pointpriors #663
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Pointpriors #663
Changes from 8 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
d05124c
implement pointwise_logpriors
bgctw 4f46102
implement varwise_logpriors
bgctw c6653b9
remove pointwise_logpriors
bgctw 216d50c
revert dot_assume to not explicitly resolve components of sum
bgctw fd8d3b2
docstring varwise_logpriores
bgctw 5842656
integrate pointwise_loglikelihoods and varwise_logpriors by pointwise…
bgctw 18beb57
record single prior components
bgctw d9945d7
forward dot_tilde_assume to tilde_assume for Multivariate
bgctw 656a757
avoid recording prior components on leaf-prior-context
bgctw 7aa9ebe
undeprecate pointwise_loglikelihoods and implement pointwise_prior_lo…
bgctw 2f67c5b
drop vi instead of re-compute vi
bgctw 9dfb9ed
include docstrings of pointwise_logdensities
bgctw c1939e0
Update src/pointwise_logdensities.jl remove commented code
bgctw 790be1d
Update src/pointwise_logdensities.jl remove commented code
bgctw 426df38
Update test/pointwise_logdensities.jl rename m to model
bgctw c32bf3b
Update test/pointwise_logdensities.jl remove unused code
bgctw 6213249
Update test/pointwise_logdensities.jl rename m to model
bgctw 3551b38
Update test/pointwise_logdensities.jl rename m to model
bgctw 95c892b
Update src/test_utils.jl remove old code
bgctw a7a7e70
rename m to model
bgctw 1653aba
JuliaFormatter
bgctw e4f0a1d
Merge branch 'pointpriors' of github.com:bgctw/DynamicPPL.jl into poi…
bgctw a99eab4
Update test/runtests.jl remove interactive code
bgctw 64ce63a
remove demo_dot_assume_matrix_dot_observe_matrix2 testcase
bgctw 456115c
ignore local interactive development code
bgctw 222529a
ignore temporary directory holding local interactive development code
bgctw 17b251a
Apply suggestions from code review: clean up comments and Imports
bgctw 7e990f0
Apply suggestions from code review: change test of applying to chains…
bgctw 8706f68
fix test on names in likelihood components
bgctw 073a325
try to fix testset pointwise_logdensities chain
bgctw 23e1711
Update test/pointwise_logdensities.jl
torfjelde 34ae4f8
Update .gitignore
torfjelde 1f251d1
Merge branch 'master' into pointpriors
torfjelde 777624a
Formtating
torfjelde 4864e60
Fixed tests
torfjelde 4d3b0c0
Updated docs for `pointwise_logdensities` + made it a doctest not
torfjelde e54fa4e
Bump patch version
torfjelde bcd82a9
Remove blank line from `@model` in doctest to see if that fixes the
torfjelde cff0941
Added doctest filter to handle the `;;]` at the end of lines for matr…
torfjelde File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
# https://invenia.github.io/blog/2022/06/17/deprecating-in-julia/ | ||
|
||
Base.@deprecate pointwise_loglikelihoods(model::Model, chain, keytype) pointwise_logdensities( | ||
model::Model, LikelihoodContext(), chain, keytype) | ||
|
||
Base.@deprecate pointwise_loglikelihoods( | ||
model::Model, varinfo::AbstractVarInfo) pointwise_logdensities( | ||
model::Model, varinfo, LikelihoodContext()) | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.