Skip to content

Conversation

@goldingn
Copy link
Member

@goldingn goldingn commented Feb 14, 2020

An interface for marginalisation of random variables

Initially just scalar discrete variables, and multivariate normal variables (via Laplace approximation).

@lintr-bot
Copy link

R/dag_class.R:6:1: style: functions should have cyclomatic complexity of less than 50, this has 51.

dag_class <- R6Class(
^

R/marginalise.R:172:1: style: Lines should not be more than 80 characters.

# x will be the target; a tf constructor function for the distribution being
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@codecov
Copy link

codecov bot commented Feb 18, 2020

Codecov Report

Merging #344 into master will increase coverage by 0.23%.
The diff coverage is 97.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #344      +/-   ##
==========================================
+ Coverage   94.94%   95.18%   +0.23%     
==========================================
  Files          30       32       +2     
  Lines        4728     5023     +295     
==========================================
+ Hits         4489     4781     +292     
- Misses        239      242       +3
Impacted Files Coverage Δ
R/dag_class.R 97.72% <100%> (+0.03%) ⬆️
R/utils.R 85.97% <95.12%> (+0.83%) ⬆️
R/marginalise.R 95.18% <95.18%> (ø)
R/marginalisers.R 98.79% <98.79%> (ø)
R/node_types.R 97.48% <0%> (+1.67%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18c581c...fe40342. Read the comment docs.

@goldingn goldingn force-pushed the feature/marginalise branch from b5989f2 to ca8920b Compare February 26, 2020 07:13
@njtierney njtierney added this to the 0.6.0 milestone Aug 20, 2024
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.

4 participants