-
Notifications
You must be signed in to change notification settings - Fork 2
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
Create Unified Hierarchy of Models and their names #91
Comments
We thought of the following structure: Taxon (lvl1) |
Whether it is important to differentiate between hierarchical and non hierarchical cfas, we are uncertain. It would also be possible to differentiate only via taxons in the struct etc./ new labels for the secondorder factors etc. |
Can you write it as:
? |
I think it makes sense to differentiate CFA types. Consider Hierachical vs Bifactor: |
Also consider: |
Very good! Now, change the Taxonomy package in a PR so that: using Taxonomy, AbstractTrees
AbstractTrees.children(d::DataType) = subtypes(d)
print_tree(Taxonomy.Taxon)
Taxon
├─ LGCM
├─ NoTaxon
├─ AbstractFactor
│ └─ Factor
└─ Pathmodel
└─ CFA is representing this structure. |
Easy right 🤣 |
'abstract type Taxon end |
Learn about CamelCase: Learn about abstract vs concrete:
https://docs.julialang.org/en/v1/manual/types/#man-abstract-types |
okay this changes things :) abstract type Taxon end |
Great, now only CamelCase: https://docs.julialang.org/en/v1/manual/style-guide/#Use-naming-conventions-consistent-with-Julia-base/ Also, I suggest renaming |
You may take insperation from other packages regarding naming, e.g., this is how stenograph looks: julia> print_tree(StenoGraphs.AbstractEdge)
AbstractEdge
├─ Edge
│ ├─ DirectedEdge
│ └─ UndirectedEdge
└─ MetaEdge
├─ Arrow
└─ ModifiedEdge
julia> print_tree(StenoGraphs.AbstractNode)
AbstractNode
├─ MetaNode
│ ├─ ModifiedNode
│ └─ ModifyingNode
└─ Node
└─ SimpleNode |
related #32 #82
The text was updated successfully, but these errors were encountered: