usamabilal / LE_Income_Inequalities_City

Income Inequalities in LE by City study
MIT License
0 stars 0 forks source link

Supplement App Main Issue: #1

Open ran-codes opened 3 years ago

ran-codes commented 3 years ago

This is the master issue to track supplement app progress. See individual issues to discuss specifics.

URL: https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City_dev/

ran-codes commented 3 years ago

@alinasmahl1 Initial draft of the app is done and ready for review or sent out to coauthors.

URL: https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City_dev/

New things/changes:

alinasmahl1 commented 3 years ago

@rl627 thank you!!!

this looks great!

I made a couple changes to the intro, appendix table 1, etc. Pushed all those changes.

ran-codes commented 3 years ago

@alinasmahl1 thanks for the changes (gj navigating the HTML file 😄 ). Changes have been pushed to app and deployed to the link above.

alinasmahl1 commented 3 years ago

Thanks!

@usamabilal want to take a look at the app?

URL: https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City_dev/

usamabilal commented 3 years ago

Looks fantastic! a few comments:

thanks!

usamabilal commented 3 years ago

Thanks Ran, this is looking great!!

I love how figure 3 looks now btw!! Thanks!

alinasmahl1 commented 3 years ago

Agree -- it looks great! I'll add the description about CBSA inclusion to the intro.

@rl627 I realized that the title for table 1 needs the word Disparities added (so it should read "Absolute and Relative Disparities in Life Expectancy in US MSAs >1 million people" -- I couldn't find the file with the figure titles, sorry!!

ran-codes commented 3 years ago

Thanks, @usamabilal for the great suggestions as always 😄 and @alinasmahl1 for editing the home page HTML file.

The new blurb was added to the main page. Figure 1 outlier dots removed (the hover effect is still there, but we cant remove that; it is just part of plotly boxplots, it always keeps the min because its part of the 5 tooltips components (max, upper fence, q3, median, q1, lower fence, min)).

alinasmahl1 commented 3 years ago

Wonderful, thank you @rl627 !! I think it's all looking great.

Is this still the appropriate link? https://drexel-uhc.shinyapps.io/LE_Income_Inequalities_City_dev/ Hoping to send the app out to the other co-authors along with the manuscript

ran-codes commented 3 years ago

Yup, this is the correct link. I guess before submission we can the URL to what ever we want. probably replace inequalities with disparities and remove dev. Or should we do that now?

alinasmahl1 commented 3 years ago

I think lets leave it for now, but we can get rid of the dev part and change to disparities before submission.