Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Philosophically Incoherent Design of "Moment" Transforms #16

Open
jacobnzw opened this issue Dec 13, 2018 · 0 comments
Open

Philosophically Incoherent Design of "Moment" Transforms #16

jacobnzw opened this issue Dec 13, 2018 · 0 comments
Labels
cleanup Something is unnecessary or inefficient

Comments

@jacobnzw
Copy link
Owner

jacobnzw commented Dec 13, 2018

StateSpaceInference sub-classes are specific to a particular density (i.e. GaussianInference, StudentInference). That is, sub-classes are designed with the Bayesian viewpoint, which imposes density on the joint p(x, z). But the same philosophy isn't followed for moment transforms!

For reasons of consistency, the same Bayesian philosophy should be reflected in the class hierarchy of the "moment" transforms (they should really be referred to as statistic transforms).

The class hierarchy might look like this:

  • GaussianTransform or GaussianStatisticTransform
    • GaussianSigmaPointTransform
  • StudentTransform or StudentStatisticTransform
    • StudentSigmaPointTransform

There might be a common base class StatisticTransform, but currently I don't see the need for it.

Consequently this will also require redesign of the StudentInference and minor changes in GaussianInference.

The following dictionary might be helpful

Bayesian world Moment world
Statistic transform Moment transform
Quadrature Sigma-point rule
PDF Moments
Gaussian filters Kalman filter
Studentian filters

Originally posted by @jacobnzw in https://github.com/_render_node/MDEyOklzc3VlQ29tbWVudDQ0MTYwMjk2OA==/timeline/issue_comment#issuecomment-441602968

@jacobnzw jacobnzw changed the title Philosophical Incoherent Design of Moment Transforms Philosophical Incoherent Design of "Moment" Transforms Dec 13, 2018
@jacobnzw jacobnzw added the cleanup Something is unnecessary or inefficient label Dec 13, 2018
@jacobnzw jacobnzw changed the title Philosophical Incoherent Design of "Moment" Transforms Philosophically Incoherent Design of "Moment" Transforms Dec 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Something is unnecessary or inefficient
Projects
None yet
Development

No branches or pull requests

1 participant