NAVADMC / ADSM

A simulation of disease spread in livestock populations. Includes detection and containment simulation.
Other
10 stars 5 forks source link

results_dailyreport #883

Closed missyschoenbaum closed 6 years ago

missyschoenbaum commented 6 years ago

@BryanHurst I can't figure out what should be going into the table Results_dailyreport. Any ideas?

If we aren't using it, can we get rid of it?

BryanHurst commented 6 years ago

As far as I can tell, this table doesn't get populated any more. Any opinion @ndh2 ?

ndh2 commented 6 years ago

I'm not sure what this table was for. It only has 2 fields "sparse_dict" and "full_line"... the word "sparse" suggests it was perhaps aiming at compressing output similar to what was done in #853? (Like the way a "sparse matrix" doesn't bother to store elements that are zero)

missyschoenbaum commented 6 years ago

Who has to get rid of it, @BryanHurst? This time we should make sure we take it out of the database template as well so it doesn't haunt me for eternity.

BryanHurst commented 6 years ago

I'll do the removal.

On Mon, Aug 6, 2018, 2:57 PM Missy Schoenbaum notifications@github.com wrote:

Who has to get rid of it, @BryanHurst https://github.com/BryanHurst? This time we should make sure we take it out of the database template as well so it doesn't haunt me for eternity.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NAVADMC/ADSM/issues/883#issuecomment-410850476, or mute the thread https://github.com/notifications/unsubscribe-auth/ABILBViOIuxhacuz9kpTh5toPHjXgJU-ks5uOK2ngaJpZM4ViKjk .

missyschoenbaum commented 6 years ago

Sold!

missyschoenbaum commented 6 years ago

Looks great. Think I need to add unit stats to the data dictionary (which is in the works and is getting double-checked at the moment).