stan-dev / stan-dev/math

Update gaussian_dlm_obs_log to use LDLT

Open
#30 3 comments 0 reactions 2 assignees View on GitHub

@syclik is already working on this.

Since Aug 17, 2016.

code cleanup distributions feature performance
Dominant language
C++
Stars
839
Forks
220
Avg merge
2d 4h
Merged PRs (30d)
14

Description

From @mbrubake on February 18, 2014 22:44

Currently still uses inverse_spd/log_determinant_spd which may be less than optimal for large systems.

Copied from original issue: stan-dev/stan#574

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.