CDCgov / cfa-viral-lineage-model

Apache License 2.0
10 stars 0 forks source link

Architecture flowchart #54

Closed swo closed 1 month ago

swo commented 2 months ago

Resolves #52

cc @afmagee42 @thanasibakis

afmagee42 commented 1 month ago

We're missing a link from "clean data" to "evaluator."

swo commented 1 month ago

We're missing a link from "clean data" to "evaluator."

good catch, I added eval data as a separate stream, separate from the training data

afmagee42 commented 1 month ago

Clarify if this reflects current reality or hoped-for reality

This is close to reality.

Figure out if we should name different components (eg with linmod.whatever classes)

Probably best not to at this point, since the boxes make conceptual sense but don't correspond neatly in the same ways to the same sorts of scripts/classes/functions.

swo commented 1 month ago

I'll move these to an issue, so we can at least get this merged and get started on it