TuringLang / DynamicPPL.jl / 11237398684

85%

main: 83%

Build Type

push

github

Committed by web-flow

Commit Message

Attempt at implementation of `VarNamedVector` (`Metadata` alternative) (#555)

* initial implementation of VarNameVector

* added some hacky getval and getdist get things to work for VarInfo

* Apply suggestions from code review

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

* added arbitrary metadata field as discussed

* renamed idcs to varname_to_index

* renamed vns to varnames for VarNameVector

* added keys impl for Metadata

* added push! and update! for VarNameVector

* added getindex_raw! and setindex_raw! for VarNameVector

* added `iterate` and `convert` (for `AbstractDict) impls for `VarNameVector`

* make the key and eltype part of the `VarNameVector` type

* added more tests for VarNameVector

* formatting

* more testing for VarNameVector

* minor changes to some comments

* added a bunch more tests for VarNameVector + several bugfixes in the process

* formatting

* added `similar` implementation for `VarNameVector`

* formatting

* removed debug statement

* made VarInfo slighly more generic wrt. underlying metadata

* fixed incorrect behavior in `keys` for `Metadata`

* minor style changes to VarNameVector tests

* style

* added testing of `update!` with smaller sizes and fixed bug related to this

* formatting

* move functionality related to `push!` for `VarNameVector` into `push!`

* Update src/varnamevector.jl

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

* several fixes to make sampling with VarNameVector + initiall tests for
sampling with VarNameVector

* VarInfo + VarNameVector tests for all demo models

* Apply suggestions from code review

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

* added docs on the design of `VarNameVector`

* Apply suggestions from code review

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

* added note on `update!`... (continued)

Run Details

557 of 663 new or added lines in 10 files covered. (84.01%)

63 existing lines in 7 files now uncovered.

3319 of 4178 relevant lines covered (79.44%)

403304.36 hits per line