ESS: Allowing diagonal covariance root with fixed indices #2283
Conversation
facebook-github-bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D55808235 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2283 +/- ##
=======================================
Coverage 99.97% 99.97%
=======================================
Files 197 197
Lines 17165 17156 -9
=======================================
- Hits 17160 17152 -8
+ Misses 5 4 -1 ☔ View full report in Codecov by Sentry. |
SebastianAment
force-pushed
the
export-D55808235
branch
from
c6d73c8 to
6847151
Compare
|
This pull request was exported from Phabricator. Differential Revision: D55808235 |
…ch#2283) Summary: This commit adds support for a diagonal covariance root in conjunction with fixed indices for ESS. This is not generally supported, as the root would have to be re-factorized. The diagonal case allows for an efficient implementation without re-factorization. Differential Revision: D55808235
SebastianAment
force-pushed
the
export-D55808235
branch
from
6847151 to
fb43775
Compare
|
This pull request was exported from Phabricator. Differential Revision: D55808235 |
SebastianAment
force-pushed
the
export-D55808235
branch
from
fb43775 to
35f561e
Compare
…ch#2283) Summary: This commit adds support for a diagonal covariance root in conjunction with fixed indices for ESS. This is not generally supported, as the root would have to be re-factorized. The diagonal case allows for an efficient implementation without re-factorization. Reviewed By: Balandat Differential Revision: D55808235
|
This pull request was exported from Phabricator. Differential Revision: D55808235 |
…ch#2283) Summary: This commit adds support for a diagonal covariance root in conjunction with fixed indices for ESS. This is not generally supported, as the root would have to be re-factorized. The diagonal case allows for an efficient implementation without re-factorization. Reviewed By: Balandat Differential Revision: D55808235
SebastianAment
force-pushed
the
export-D55808235
branch
from
35f561e to
036c5c1
Compare
|
This pull request was exported from Phabricator. Differential Revision: D55808235 |
|
This pull request has been merged in 7c28a20. |
2 participants
Summary: This commit adds support for a diagonal covariance root in conjunction with fixed indices for ESS. This is not generally supported, as the root would have to be re-factorized. The diagonal case allows for an efficient implementation without re-factorization.
Differential Revision: D55808235