Skip to content

Conversation

@torfjelde
Copy link
Member

@torfjelde torfjelde commented Aug 8, 2023

These tests pass nicely locally. Feel free to just merge this into your branch @yebai if you're happy with the changes!

@torfjelde torfjelde requested a review from yebai August 8, 2023 21:43
@torfjelde torfjelde mentioned this pull request Aug 8, 2023
@yebai
Copy link
Member

yebai commented Aug 8, 2023

Thanks @torfjelde!

@yebai yebai merged commit 184888a into yebai-patch-1 Aug 8, 2023
@yebai yebai deleted the torfjelde/516-tests branch August 8, 2023 21:45
github-merge-queue bot pushed a commit that referenced this pull request Aug 9, 2023
* Bugfix in `VarInfo`.

* Update Project.toml

* Update src/varinfo.jl

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

* Update src/varinfo.jl

Co-authored-by: David Widmann <[email protected]>

* Added tests.

* Apply suggestions from code review

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

* Fix for tests in #516 (#517)

* fixed tests for linking of dirichlet with different dimensionality

* added usage of same logp in TestUtils.setup_varinfos

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: David Widmann <[email protected]>
Co-authored-by: Tor Erlend Fjelde <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants