Analyticsphere / metricsReportsRequests

Used to provide issue tracking for changes and additions to the Connect Metrics reporting.
MIT License
0 stars 0 forks source link

Weekly Survey Metrics Report Revamp #172

Open robertsamm opened 1 week ago

robertsamm commented 1 week ago

We would like to revamp the Weekly Survey Metrics Report now that we have completed the Ops Report and have added more metrics to the report for recently released surveys. Please see below. This is a lower priority request, higher priority items should take precedent.

General Report Flow Updates

Updates to current metrics

New Metrics to Be Added

KELSEYDOWLING7 commented 6 days ago

Hi Amelia, thanks for making this checklist! It makes it MUCH easier to keep track of everything. I had some down time and these were all pretty quick changes.

Please let me know if I misinterpreted any of the new requests. I made the assumption that number 14 should only be out of those who completed CES, and that the rows in number 15 should be those verified at those dates: 'More than 1 Year' (~July 2021-Dec 2022), '6 Months - 1 Year' (~Jan 2023-June 2023), '3 Months - 6 Months' (~July 2023-Dec 2023), 'At 3 Months' (anyone prospective). Also we already had a short intro just to explain the module acronyms. Please let me know if you want that kept, removed, or moved. A good alternative spot might be under the "Baseline Survey" header.

CCC-Weekly-Module-Metrics_RMD.pdf

robertsamm commented 6 days ago

Hi @KELSEYDOWLING7 yay! I had noticed you made the checkmarks on a recent github issue so thought it might be easier to do the same thing here! And wow, that was fast! This is looking great - I just have a few small edits:

KELSEYDOWLING7 commented 5 days ago

Good morning, few questions/comments. Table 5.8 already has the abbreviations for CES so I believe we should be good there. Then, I'm a bit confused at what the new title should be for 4.10. Can you clarify specifically what you'd like the title and category row label to be?

robertsamm commented 5 days ago

Good morning @KELSEYDOWLING7!

For table 5.8 - it should have the same changes applied in number 6/table 4.11 where the category row label would state 'Survey Completed After CES' and then the survey labels in the rows are shortened to just refer to the survey itself and drop 'completed after CES' in each label

For Table 4.10 - Just need to add another 'QOL' into the title and label so Title: QOL Survey Completion By Time from Verification to QOL Survey Release Row label: Time from Verification to QOL Survey Release

KELSEYDOWLING7 commented 5 days ago

Sorry, maybe I'm missing something? I think 5.8 already has all that: image

CCC-Weekly-Module-Metrics_RMD.pdf

robertsamm commented 5 days ago

Hi @KELSEYDOWLING7 I was envisioning something more like below for Table 4.11 and 5.8. What do you think? That way the 'completed after QOL/CES' doesn't have to be repeated in each row.

<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns="http://www.w3.org/TR/REC-html40">

Table 4.11. Baseline Survey Completion On Day of or After QOL Survey Completion |   -- | -- Survey Completed After QOL | N(%) BOH | N(%) MRE | N(%) SAS | N(%) LAW | N(%) BUM | N(%) BU | N(%) MW | N(%) COVID | N(%) None | N(%) Total Submitted QOL Survey | N(%)

robertsamm commented 5 days ago

Forgot to add, I still need to review the changes to the rest of the report which I will do shortly!

brotzmanmj commented 5 days ago

One small request, can you include a note somewhere (the summary on the first page or in the Initial Survey section, a note to indicate the date when all four modules were in MyConnect?

KELSEYDOWLING7 commented 5 days ago

Sure thing, do we have that date anywhere?

robertsamm commented 5 days ago

Yes, July 5th, 2022 is when all 4 sections were available in prod

KELSEYDOWLING7 commented 5 days ago

How does this sound:

Starting July 5th, 2022, all four Initial Survey Modules were in MyConnect. In order of how the sections appear on MyConnect, the Initial Survey Sections are:

BOH= Background and Overall Health (Module 1)

LAW= Where you Live and Work (Module 4)

MRE= Medications, Reproductive Health, Exercise, and Sleep (Module 2)

SAS= Smoking, Alcohol, and Sun Exposure (Module 3)

robertsamm commented 5 days ago

@KELSEYDOWLING7 yes, I think that sounds perfect!

robertsamm commented 5 days ago

Hi @KELSEYDOWLING7 final suggested edits hopefully!

KELSEYDOWLING7 commented 5 days ago

So do we want to remove the mention of the Menstrual cycle survey from these tables as well? The other biospecimen surveys (BU, BUM, and COVID) are on there too. Should those be removed? image

If not, please see below:

CCC-Weekly-Module-Metrics_RMD.pdf

robertsamm commented 5 days ago

@KELSEYDOWLING7 no, I would leave the mention of the menstrual cycle and covid surveys in those metrics but for the one metric that is specific to menstrual cycle only that one can be moved to the biospec weekly report

KELSEYDOWLING7 commented 5 days ago

Got it, ok that table/section was removed in that last version I just posted

robertsamm commented 5 days ago

Thanks @KELSEYDOWLING7 I just reviewed the latest version and it looks great! The footnotes need to be added in to the age/race/sex tables 1.9-1.11. After that, this looks perfect and is ready for prod on Monday!

KELSEYDOWLING7 commented 4 days ago

Oh weird! The footnotes were all in the code but weren't rendering. That's resolved now, please let me know if we're good to close the issue. We're also updating the report automation this afternoon for Monday. CCC-Weekly-Module-Metrics_RMD.pdf