gowthamrao
commented
1 year ago i suggest the following default selections for age and sex
Open pbr6cornell opened 1 year ago
gowthamrao
commented
1 year ago i suggest the following default selections for age and sex
gowthamrao
commented
1 year ago I would presume this should be % of persons in database that belong to cohort, so definitely isn't 100%.
This has been an issue since beginning, not unique to CD v3. However, @azimov we can remediate this in CD v3 - because we have persons in database in the metadata output (new to version 3).
gowthamrao
commented
1 year ago This code, introduced in version 3 gives us datasource summary that maybe useful here
azimov
commented
1 year ago 1, 2, and 4 are fixed. 3 Is a more involved issue
ENHANCEMENT: Remove all reactive dropdown/checkboxes. Instead, make user pick the desired parameters and then click 'Generate plot'. Currently, with the reactives, each edit to age group, calendar year filter results in refresh, and can queue up and take awhile.
BUG; Download button to the right of the user selections does not work. Not sure if it should produce jpg of graph or tabular output, but currently it generates dowload.html of page that is nonsense.
POTENTIAL BUG? The hoverover/tooltip show Incidence Proportion as 100% for all cohorts. I would presume this should be % of persons in database that belong to cohort, so definitely isn't 100%. But, it seems like it may be % of persons in cohort who belong to a specific age/sex strata? I don't think thats the desired/expected behavior, but if it is, then should be clearly documented somewhere to avoid confusion.
ENHANCEMENT: Currently, cohorts are labeled by Cohort field, with a 'lookup' above the graph. It'd be desirable to directly apply the cohort name on the graph (ex. as column heading across trellis.).