Remove get_defaults

Author: jlubken

Dockerfile

@@ -1,16 +1,17 @@
 FROM python:3.7.7-slim-buster
-RUN mkdir /app
+ENV PARAMETERS=./defaults/webapp.cfg
 WORKDIR /app
+RUN mkdir /app
+# Creating an empty src dir is a (hopefully) temporary hack to improve layer caching and speed up image builds
+# todo fix once the Pipfile, setup.py, requirements.txt, pyprojec.toml build/dist story is figured out
+RUN mkdir /src
 COPY README.md .
 COPY setup.cfg .
 COPY setup.py .
-COPY requirements.txt .
-# Creating an empty src dir is a (hopefully) temporary hack to improve layer caching and speed up image builds
-# todo fix once the Pipfile, setup.py, requirements.txt, pyprojec.toml build/dist story is figured out
-RUN mkdir src && pip install -q .
 COPY .streamlit .streamlit
-COPY settings.cfg .
+COPY defaults defaults
 COPY src src
+RUN pip install -q .
 
 CMD ["streamlit", "run", "src/app.py"]
 

Dockerfile.dash

@@ -4,11 +4,14 @@ WORKDIR /code
 
 ARG PORT
 ENV PORT $PORT
+ENV PARAMETERS=./defaults/webapp.cfg
 
-COPY requirements.txt requirements.txt
-RUN pip install -r requirements.txt
-
+COPY README.md .
+COPY setup.py .
+COPY setup.cfg .
+COPY defaults defaults
 COPY src src 
+RUN pip install -q .
 
 # EXPOSE $PORT 
 

settings.cfg renamed to defaults/cli.cfg

@@ -1,13 +1,14 @@
---current-hospitalized 14
---doubling-time 4.0
+--current-hospitalized 69
+--doubling_time 4.0
 --hospitalized-days 7
 --hospitalized-rate 0.025
 --icu-days 9
 --icu-rate 0.0075
 --infectious-days 14
 --market_share 0.15
---n-days 60
---population 4119405
+--n-days 100
+--population 3600000
+--recovered 0
 --relative-contact-rate 0.3
 --ventilated-days 10
 --ventilated-rate 0.005

defaults/readme.txt

@@ -0,0 +1,3 @@
+Webapps need default values for both date_first_hosptalized and doubling_time. The ui overrides one of them.
+
+CLI needs either date_first_hospitalized xor doubling_time.

defaults/webapp.cfg

@@ -0,0 +1,15 @@
+--current-hospitalized 69
+--date-first-hospitalized 2020-03-07
+--doubling-time 4.0
+--hospitalized-days 7
+--hospitalized-rate 0.025
+--icu-days 9
+--icu-rate 0.0075
+--infectious-days 14
+--market_share 0.15
+--n-days 100
+--population 3600000
+--recovered 0
+--relative-contact-rate 0.3
+--ventilated-days 10
+--ventilated-rate 0.005

setup.py

@@ -1,6 +1,6 @@
 """Setup file for chime
 """
-__version__ = "1.1.2"  # update VERSION in constants.py
+__version__ = "1.1.3"  # update VERSION in constants.py
 __author__ = "Predictive Healthcare @ Penn Medicine"
 
 from os import path

src/app.py

@@ -1,16 +1,18 @@
 """App."""
 
+import os
+
 import altair as alt  # type: ignore
 import streamlit as st  # type: ignore
 
+from penn_chime.parameters import Parameters
 from penn_chime.presentation import (
     display_download_link,
     display_footer,
     display_header,
     display_sidebar,
     hide_menu_style,
 )
-from penn_chime.settings import get_defaults
 from penn_chime.models import SimSirModel
 from penn_chime.charts import (
     build_admits_chart,
@@ -26,7 +28,7 @@
 # In dev, this should be shown
 st.markdown(hide_menu_style, unsafe_allow_html=True)
 
-d = get_defaults()
+d = Parameters.create(os.environ, [])
 p = display_sidebar(st, d)
 m = SimSirModel(p)
 

src/chime_dash/__init__.py

@@ -4,10 +4,13 @@
 dash instance defined here
 """
 
-from dash import Dash
+import os
 from typing import TypeVar
+
+from dash import Dash
+from penn_chime.parameters import Parameters
+
 from chime_dash.app.config import from_object
-from penn_chime.settings import get_defaults
 from chime_dash.app.pages.root import Root
 from chime_dash.app.utils.callbacks import wrap_callbacks
 
@@ -31,7 +34,10 @@ def create_app(context: str = 'prod') -> DashAppInstance:
     Env = from_object(context)
 
     LANGUAGE = Env.LANG
-    body = Root(LANGUAGE, get_defaults())
+    body = Root(
+        LANGUAGE,
+        Parameters.create(os.environ, []),
+    )
 
     App = Dash(
         __name__,

src/chime_dash/app/components/base.py

@@ -12,12 +12,9 @@
 from dash_html_components import Div
 
 from penn_chime.parameters import Parameters
-from penn_chime.settings import get_defaults
 
 from chime_dash.app.utils.templates import read_localization_yml, read_localization_markdown
 
-DEFAULTS = get_defaults()
-
 
 class Component(ABC):
     """Base component for rendering dash html objects and registering callbacks
@@ -32,7 +29,7 @@ class Component(ABC):
     external_stylesheets: List[str] = []
     external_scripts: List[str] = []
 
-    def __init__(self, language: str = "en", defaults: Parameters = DEFAULTS):
+    def __init__(self, language: str = "en", defaults: Parameters = None):
         """Initializes the component
         """
         self.language = language
@@ -94,7 +91,7 @@ def content(self) -> Union[str, Dict[str, str], None]:
 class Page(Component):
     callbacks_cls = None
 
-    def __init__(self, language: str = "en", defaults: Parameters = DEFAULTS):
+    def __init__(self, language: str = "en", defaults: Parameters = None):
         super().__init__(language, defaults)
         self.callbacks_cls(self)
 

src/chime_dash/app/components/navbar.py

@@ -11,17 +11,14 @@
 from chime_dash.app.components.menu import Menu
 
 from penn_chime.parameters import Parameters
-from penn_chime.settings import get_defaults
-
-DEFAULTS = get_defaults()
 
 
 class Navbar(Component):
     """Navigation bar contains menu and brand
     TODO refactor / design input on style and layout
     """
 
-    def __init__(self, language: str = "en", defaults: Parameters = DEFAULTS):
+    def __init__(self, language: str = "en", defaults: Parameters = None):
         """Sets up self, menu and header
         """
         super().__init__(language, defaults=defaults)