Skip to content

Commit bb81918

Browse files
committed
Incorporate suggestions from review
1 parent 41f67fb commit bb81918

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

src/threadsafe.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ ThreadSafeVarInfo(vi::ThreadSafeVarInfo) = vi
1717
# just update the array of log probabilities.
1818
function acclogp!(vi::ThreadSafeVarInfo, logp)
1919
vi.logps[Threads.threadid()] += logp
20-
return getlogp(vi)
20+
return vi
2121
end
2222

2323
# The current log probability of the variables has to be computed from

src/varinfo.jl

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -602,15 +602,21 @@ getlogp(vi::AbstractVarInfo) = vi.logp[]
602602
Set the log of the joint probability of the observed data and parameters sampled in
603603
`vi` to `logp`.
604604
"""
605-
setlogp!(vi::VarInfo, logp) = vi.logp[] = logp
605+
function setlogp!(vi::VarInfo, logp)
606+
vi.logp[] = logp
607+
return vi
608+
end
606609

607610
"""
608611
acclogp!(vi::VarInfo, logp)
609612
610613
Add `logp` to the value of the log of the joint probability of the observed data and
611614
parameters sampled in `vi`.
612615
"""
613-
acclogp!(vi::VarInfo, logp) = vi.logp[] += logp
616+
function acclogp!(vi::VarInfo, logp)
617+
vi.logp[] += logp
618+
return vi
619+
end
614620

615621
"""
616622
resetlogp!(vi::AbstractVarInfo)

0 commit comments

Comments
 (0)