Skip to content
Snippets Groups Projects

Fix for MultivariateGaussianDist for 1d cases

Merged Fix for MultivariateGaussianDist for 1d cases
matthew-pitkin/bilby:mvn_1d_fix into master
Merged Matthew David Pitkin requested to merge matthew-pitkin/bilby:mvn_1d_fix into master

If using the MultivariateGaussianDist to set a one dimensional prior, e.g., using it to set a multi-modal distribution on a single parameter, the sample method currently fails. This MR fixes the failure by making sure that the samples that are created are 1d arrays.

Merge request reports

Pipeline #78195 passed

Stage: test

Pipeline passed for f42628ce on matthew-pitkin:mvn_1d_fix

Approved by
Colm Talbot Gregory Ashton

Merged by Gregory AshtonGregory Ashton 5 years ago (Sep 3, 2019 2:59pm UTC)

Merge details

  • Changes merged into master with 19fddb2f.
  • Deleted the source branch.

Pipeline #78203 passed with warnings

Stage: test
Stage: deploy

Pipeline passed with warnings for 19fddb2f on master

Test coverage 70.00% from 0 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply