Jcunni/sprucing up #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lpanda2
approved these changes
May 22, 2023
pages/drilldown.py
Outdated
| st.markdown("""Inpatient and Outpatient spend are the largest drivers of PMPM during this time period""") | ||
| plost.bar_chart(data=grouped_pmpm, bar='Metric', value=pmpm_cats, stack='normalize', | ||
| direction='horizontal', legend='top', height=200) | ||
| st.markdown("### Total Pharmacy Spend Over Claim Period**") |
There was a problem hiding this comment.
quick nit there's a ** at the end of this
pages/drilldown.py
Outdated
| plost.bar_chart(data=grouped_pmpm, bar='Metric', value=pmpm_cats, stack='normalize', | ||
| direction='horizontal', legend='top', height=200) | ||
| st.markdown("### Total Pharmacy Spend Over Claim Period**") | ||
| st.markdown("""Pharmacy Spend appears largely steady between 2016 and 2018, averaging between $100k-$200k a month. |
There was a problem hiding this comment.
$100k-$200k <- try wrapping this in backticks or separating with a space because it's inferring as latex
main_page.py
Outdated
Comment on lines
75
to
78
| col1, col2, col3 = st.columns([1,1,1]) | ||
| col1.metric("Total Medical Spend", '${}'.format(util.human_format(total_spend))) | ||
| col2.metric("Total Member Months", util.human_format(total_member_months)) | ||
| col3.metric("Average PMPM", '${}'.format(util.human_format(avg_pmpm))) |
There was a problem hiding this comment.
This is awesome -- I'll copy this into the financial breakdown. Should we add pharmacy spend?
lpanda2
approved these changes
May 22, 2023
jcunninghame
added a commit
that referenced
this pull request
Jun 21, 2023
* initial page with a data table from the google sheet, start reqs.txt for later use * some manual data cleaning, throw in a line chart with Y axis select * gitignore and some requirement updates * snowflake connect, use .env vars * update SQL query, add in time filter * some summary metrics * some page 2 work * patient demo charts * some header updates, container sizing, pre-commit req * more ignoring * [ADDED] a small util file * [ADDED] a Financial Summary page * [ADDED] some basic summary * [ADDED] my original cost summary * [FIXED] sort the condition table * [ADDED] a little visual delighter * cached data funcs, use util, lowercasing * blurbs * cache and func treatment for data, adding some blurbs * pretty formating * page rename * small errs * pharm spend * [RENAMED] files and then removed chronic conditions from it * [ADDED] placeholder for the components * [ADDED] the following: - claim type chart breakdown (with pharmacy spend) - filter by year month for the service category 2 - improve label/axis in chart * [ADDED] chronic condition charts * wip * financial ban work, incorporate * ban component use, make slider a year filter * copy condition graph over * variety * big stacked horizontal bar * pharm trend line, remove drilldown page * config for pre-commit * the great reformatting * dep * moving pharmacy block * initial readme * use lipsas database for now * wip, animation working for altair chart * echart line w/ animation, some state management * colors and turn off animation for now * switch from dotenv to tomli * custom theme * [ADDED] test results * [ADDED] use case * remove unused deps * pandas version of snowflake connector * [ADDED] Pop * [FIXED] think this slider wasn't working properly? * move chart into components file, grouped bar for pop comparison * move data calls to one file * [REMOVED] charts and unused text * [BUGFIX] hook claim type pmpm data up to time slider * [ADDED] a color palette and some styling on the metric cards * [ADDED] light color and styling to this line chart * [ADDED] Logo * [ADDED] chatgpt language * [FIXED] round the numbers and add the new color formats * [ADDED] colors and highlighting to the other charts - also rounded values in the pmpm claim type line chart * [ADDED] show the table and the chart as tabs * cluster of drilldowns * stylizing, make it wide mode * tab branding * renaming and organizing pages, adjusting chart heights * read in data from s3 extracts instead of snowflake * fix year * hide some pages we dont want to show, renaming, update deps * missing dep for deployment * case sesentive * hide other pages, add logo to top of financial summary * [ADDED] threshold for row count * handle one year of data better * change from year to quarter * query updates, point to lds data * wip * get provider grouping working again * comment out cost summary because not possible on their source data right now * copy updates * chart height --------- Co-authored-by: Lipsa Panda <lpanda2014@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Code cleanup and utilizing the util functions Lipsa added with her last PR. Also some blurbs about data to make things a data story?