TuringLang / DynamicPPL.jl / 12200518816

85%

main: 83%

Build Type

push

github

Committed by web-flow

Commit Message

Introduce `vector_getrange` and `vector_getranges` for `VarInfo` (#738)

* replaced a closure with `Fix1`

* added correct implementation of `getrange` for `TypedVarInfo`

* fixed calls  to varinfo methods which should be metadata methods

* fixed typo

* use `setval!` on the metadata directly instead of on the varinfo

* added `length` implementation for `VarInfo` and `Metadata`

* added testing for `getranges

* introduce `vector_length` instead of `length`, since `length` already
refers to the dictionary-like length impl, not vector-like

* fixed bug in `getranges` for untyped varinfo

* added proper testing for other `VarInfo` types

* bump patch version

* separated the `getrange` version which returns the range of the vecto
representaiton rather than the internal representaiton into
`vector_getrange` to make its function explicit

* formatting

* removed `vector_getrange` for metadata

* added handling of missing indices + tests for these cases

* added handling of duplicated values

* removed no-longer relevant comment

* fixed impl of `vector_getrange` and `vector_getranges` for threadsafe varinfo

* fixed `vector_getranges` when `vns` are not found

Run Details

33 of 45 new or added lines in 3 files covered. (73.33%)

2 existing lines in 1 file now uncovered.

3668 of 4249 relevant lines covered (86.33%)

1150254.06 hits per line