-
Notifications
You must be signed in to change notification settings - Fork 2
Closed
Description
The Abstract prefix came up in #1146
Listing current abstract types for "do we use Abstract prefix?":
AbstractType | JustType |
---|---|
AbstractDFG | |
AbstractParams | |
AbstractBlobstore | |
DFGNode | |
AbstractDFGVariable | |
AbstractDFGFactor | |
AbstractPackedFactorObservation | |
AbstractFactorObservation | |
AbstractPrior | |
AbstractRelative | |
FactorSolverCache | |
VariableStateType | |
PackedBelief | |
AbstractPointParametricEst | will be obsolete |
Note: DFGVariable and DFGFactor conflicts with names to be deprecated and is too similar to VariableDFG.
Julia uses the Abstract prefix regularly, and it's a recommendation in the SciML style guide: https://docs.sciml.ai/SciMLStyle/stable/#General-Naming-Principles
DFG v1.0 implementation :
Old | AbstractType | Alias |
---|---|---|
AbstractDFG | -- | |
AbstractParams | AbstractDFGParams | DFGParams |
AbstractBlobstore | Blobstore | |
DFGNode | AbstractGraphNode | GraphNode |
AbstractDFGVariable | AbstractGraphVariable | GraphVariable |
AbstractDFGFactor | AbstractGraphFactor | GraphFactor |
AbstractPackedFactorObservation | AbstractPackedObservation | PackedObservation |
AbstractFactorObservation | AbstractObservation | Observation |
AbstractPrior | AbstractPriorObservation | PriorObservation |
AbstractRelative | AbstractRelativeObservation | RelativeObservation |
FactorSolverCache | AbstractFactorCache | FactorCache |
VariableStateType | AbstractStateType | StateType |
AbstractPackedBelief | PackedBelief |