-
Notifications
You must be signed in to change notification settings - Fork 8
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
TagBot trigger issue #15
Comments
Triggering TagBot for merged registry pull request: JuliaRegistries/General#32616 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#33743 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#33811 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#41455 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#44580 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#48304 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#49505 |
This is a draft PR introducing a `Model` type that stores and makes use the model graph. The main type introduced here is the `Model` struct which stores the `ModelState` and `DAG`, each of which are their own types. `ModelState` contains information about the node values, dependencies and eval functions and `DAG` contains the graph and topologically ordered vertex list. A model can be constructed in the following way: ```julia julia> nt = ( s2 = (0.0, (), () -> InverseGamma(2.0,3.0), :Stochastic), μ = (1.0, (), () -> 1.0, :Logical), y = (0.0, (:μ, :s2), (μ, s2) -> MvNormal(μ, sqrt(s2)), :Stochastic) ) (s2 = (0.0, (), var"#33#36"(), :Stochastic), μ = (1.0, (), var"#34#37"(), :Logical), y = (0.0, (:μ, :s2), var"#35#38"(), :Stochastic)) julia> Model(nt) Nodes: μ = (value = 1.0, input = (), eval = var"#16#19"(), kind = :Logical) s2 = (value = 0.0, input = (), eval = var"#15#18"(), kind = :Stochastic) y = (value = 0.0, input = (:μ, :s2), eval = var"#17#20"(), kind = :Stochastic) DAG: 3×3 SparseMatrixCSC{Float64, Int64} with 2 stored entries: ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ 1.0 1.0 ⋅ ``` At present, only functions needed for the constructors are implemented, as well as indexing using `@varname`. I still need to complete the integration with the AbstractPPL api. TODO: ~~- [ ] `condition`/`decondition`,~~ ~~- [ ] `sample`~~ ~~- [ ] `logdensityof`~~ - [x] pure functions for ordered dictionary, as outlined in [AbstractPPL](https://github.com/TuringLang/AbstractPPL.jl#property-interface) Feedback on `Model` structure welcome whilst I implement the remaining features!
Triggering TagBot for merged registry pull request: JuliaRegistries/General#55582 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#55608 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#56246 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#73779 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#74778 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#75504 |
Triggering TagBot for merged registry pull request This extra notification is being sent because I expected a tag to exist by now, but it doesn't. |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#77381 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#78510 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#94989 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#102538 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#103811 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#104445 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#104544 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#104783 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#105113 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#116385 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#121651 |
Triggering TagBot for merged registry pull request: JuliaRegistries/General#121704 |
This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your
TagBot.yml
to include issue comment triggers.Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment
TagBot fix
on this issue.I'll open a PR within a few hours, please be patient!
The text was updated successfully, but these errors were encountered: