Open alinasmahl1 opened 2 years ago
O sorry, one more thing. For the map, we've added in Maine and Wisconsin. Just a note for updating that map.
andd one more- can we please add the BGV to table 1 as well?
@alinasmahl1 updates made (as per the checklist above) please see the link below to review.
URL: https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City
(@alinasmahl1 , remmeber to change the link in the paper, it says _dev). And we will need a few changes to table/figure numbers if you agree with my latest paper changes.
Looks great @rl627 , agree you hit all the checklist items. A couple of additions I noticed.
Thanks @usamabilal @alinasmahl1 for the comments!! 😄
Updates made: https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City/
@alinasmahl1 , do you think a slider to limit population size would be beneficial for the reader? some may want to explore different sorting of the 1million people thing. Maybe a double slider so people can indicate min and max?
Thank you @rl627 it looks fantastic.
@usamabilal hmm while I think that would be useful, we already have a lot of options for different toggles, and I think we'd need to then explain the different optionality in the article text, and I worry that might complicate things more (and it's already got a lot of alternative views of the data). So I think I'd suggest we don't include a slider.
Ok!
From: alinasmahl1 @.> Sent: Tuesday, May 24, 2022 8:47:52 PM To: usamabilal/LE_Income_Inequalities_City @.> Cc: Bilal,Usama @.>; Mention @.> Subject: Re: [usamabilal/LE_Income_Inequalities_City] Updates to Dashboard (Issue #9)
External.
@usamabilalhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusamabilal&data=05%7C01%7Cub45%40drexel.edu%7Cf91a7838d8a0426cd26708da3de83363%7C3664e6fa47bd45a696708c4f080f8ca6%7C0%7C0%7C637890364754968385%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=GQwxlPoqL3EParluqyBiCcrqoy14Z0U0HNg0IVxop1o%3D&reserved=0 hmm while I think that would be useful, we already have a lot of options for different toggles, and I think we'd need to then explain the different optionality in the article text, and I worry that might complicate things more (and it's already got a lot of alternative views of the data). So I think I'd suggest we don't include a slider.
— Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusamabilal%2FLE_Income_Inequalities_City%2Fissues%2F9%23issuecomment-1136570371&data=05%7C01%7Cub45%40drexel.edu%7Cf91a7838d8a0426cd26708da3de83363%7C3664e6fa47bd45a696708c4f080f8ca6%7C0%7C0%7C637890364754968385%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=44X%2F%2F1hOa2HJ6cJ1dkPBXS4uJuKrBjO2FqkxzG1ZHTo%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB3JZYHTMQLSSBINRPSZS7DVLV2DRANCNFSM5WJYEYMA&data=05%7C01%7Cub45%40drexel.edu%7Cf91a7838d8a0426cd26708da3de83363%7C3664e6fa47bd45a696708c4f080f8ca6%7C0%7C0%7C637890364754968385%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=HXFd9m45qc9A0COe4wy1iJ%2F0HqqeiAI73fBsfh4KNgQ%3D&reserved=0. You are receiving this because you were mentioned.Message ID: @.***>
@rl627 can you please update the figures on the dashboard so that they now use the median estimates? The code for the figures pulls through the medians, so shouldn't take much to update. Note, we've added one more metric - the between group variance. Can you please add this to figure 1?
For Table 1, can we please also include the 2.5 and 97.5% in the table? So repeat the main text Table 1, but for all MSAs (not just those with pops>1 million) and still include all the interactive features you've already created.
Can you please also add a toggle so figures can be shown for life expectancy at birth, age 25 or age 65 for figures 1 & 3?
Can we also update the text about what the article does- so that we remove "2) extends visualizations to all US MSAs including those with populations of less than 1 million residents" to "extends Table 1 to all US MSAs including those with populations of less than 1 million resident, and 3) allows exploration of the Figure 1 using all 10 disparity measures."
Please let me know if there's any q's.