Skip to content

Commit f229dec

Browse files
Merge branch 'develop' into issues/66
2 parents 46bde7b + 7f7e33e commit f229dec

39 files changed

+3020
-624
lines changed

.cache/v/cache/lastfailed

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"test_app.py": true
3+
}

.dockerignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,3 +25,4 @@ venv
2525
ENV
2626
env.bak
2727
venv.bak
28+
cypress

.github/workflows/pythonapp.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,11 @@ jobs:
3737
run: |
3838
pip install pytest
3939
python -m pytest
40+
- name: Run App
41+
run: |
42+
streamlit run src/app.py &
43+
- name: Cypress
44+
uses: cypress-io/[email protected]
45+
with:
46+
working-directory: e2e
47+

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,4 @@ RUN pip install -q -r requirements.txt
1010

1111
COPY . ./
1212

13-
CMD ["streamlit", "run", "app.py"]
13+
CMD ["streamlit", "run", "src/app.py"]

Procfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
web: STREAMLIT_SERVER_PORT=$PORT streamlit run app.py
1+
web: STREAMLIT_SERVER_PORT=$PORT streamlit run src/app.py

README.md

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,24 @@
11
# CHIME
22

3-
The **C**OVID-19 **H**ospital **I**mpact **M**odel for **E**pidemics
3+
The **C**OVID-19 **H**ospital **I**mpact **M**odel for **E**pidemics ([penn-chime.phl.io](http://penn-chime.phl.io/))
44

55
[![CHIME](https://user-images.githubusercontent.com/1069047/76693244-5e07e980-6638-11ea-9e02-1c265c86fd2b.gif)](http://predictivehealthcare.pennmedicine.org/chime)
66

7-
Try it out online at [penn-chime.phl.io](http://penn-chime.phl.io/)
7+
## Background
8+
The [CHIME](https://penn-chime.phl.io/) (COVID-19 Hospital Impact Model for Epidemics) Application is designed to assist hospitals and public health officials with understanding hospital capacity needs as they relate to the COVID pandemic. CHIME enables capacity planning by providing estimates of total daily (i.e. new) and running totals of (i.e. census) inpatient hospitalizations, ICU admissions, and patients requiring ventilation. These estimates are generated using a [SIR (Susceptible, Infected, Recovered)](https://mathworld.wolfram.com/SIRModel.html) model, a standard epidemiological modeling technique. Our model has been validated by several epidemiologists including [Michael Z. Levy, PhD](https://www.dbei.med.upenn.edu/bio/michael-z-levy-phd), Associate Professor of Epidemiology, Department of Biostatistics, Epidemiology and Informatics at the Perelman School of Medicine.
89

9-
Interested in [contributing](https://codeforphilly.github.io/chime/CONTRIBUTING.html)?
10+
Originally developed in `github.com/pennsignals/chime`, active development is now at `github.com/pennsignals/chime`.
1011

11-
Join our [Code For Philly](https://codeforphilly.org/projects/chime) project or our [Slack workspace](https://codeforphilly.org/chat) in the [#covid19-chime-penn](https://codeforphilly.org/chat/covid19-chime-penn) channel if you'd like to chat with us. We'd appreciate your [feedback](http://predictivehealthcare.pennmedicine.org/contact/).
12+
### Documentation
13+
14+
- [User Documentation](https://code-for-philly.gitbook.io/chime/)
15+
- [Developer Documentation](https://codeforphilly.github.io/chime/)
1216

13-
## Documentation
17+
## Contributing
1418

15-
- [Getting Started: Try Online](https://codeforphilly.github.io/chime/getting-started/try-online.html)
16-
- [Getting Started: Run Locally](https://codeforphilly.github.io/chime/getting-started/run-locally.html)
1719
- [Contributing: Overview](https://codeforphilly.github.io/chime/CONTRIBUTING.html)
18-
- [Contributing: Application Development](https://codeforphilly.github.io/chime/contributing/app-dev.html)
20+
21+
Join our [Code For Philly](https://codeforphilly.org/projects/chime) project or our [Slack workspace](https://codeforphilly.org/chat) in the [#covid19-chime-penn](https://codeforphilly.org/chat/covid19-chime-penn) channel if you'd like to chat with us. We'd appreciate your [feedback](http://predictivehealthcare.pennmedicine.org/contact/).
1922

2023
## Thanks
2124

app.py

Lines changed: 0 additions & 216 deletions
This file was deleted.

docs/CONTRIBUTING.md

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -4,30 +4,30 @@ Welcome, grab a cup of coffee, and let's roll our sleeves up! By participating i
44

55
## What should I work on?
66

7-
As of March 19, help is especially wanted in these areas:
8-
9-
- **Python wizards** for application development ([#chime-app](https://codeforphilly.org/chat?channel=chime-app))
10-
- Hardening the code
11-
- Making it more flexible
12-
- Unit tests!
13-
- **DevOps heroes** for application hosting ([#chime-ops](https://codeforphilly.org/chat?channel=chime-ops))
14-
- Maintaining Penn's instance
15-
- Ensuring easy redeploys
16-
- **Math and stats friends** for model improvements ([#chime-analysis](https://codeforphilly.org/chat?channel=chime-analysis))
17-
- **Consulting DevOps champs** to help other public agencies set up their own instances ([#chime-help](https://codeforphilly.org/chat?channel=chime-help)
7+
Our project work is organized by 4 main areas:
8+
9+
| Area | What type of help wanted? | Project Board | Slack channel | Maintainers |
10+
| --- | --- | --- | --- | --- |
11+
| App Platform | Python wizards to work on application development, hardening the code, making it more flexible, and unit testing | [Board](https://github.com/CodeForPhilly/chime/projects/3) | [#chime-app](https://codeforphilly.org/chat?channel=chime-app) | @quinndougherty |
12+
| Analysis | Epidemiologists and other math and stats friends, for model improvements | [Board](https://github.com/CodeForPhilly/chime/projects/5) | [#chime-analysis](https://codeforphilly.org/chat?channel=chime-analysis) | @sam-qordoba, @cjbayesian |
13+
| DevOps | DevOps heroes (especially if you have Kubernetes experience) for hosting, maintenance, and ensuring easy redeploys. | [Board](https://github.com/CodeForPhilly/chime/projects/2) | [#chime-ops](https://codeforphilly.org/chat?channel=chime-ops) | @moosequest, @themightychris, @lottspot |
14+
| General Project Ops | Writing, documentation, and project and product management | [Board](https://github.com/CodeForPhilly/chime/projects/6) | | @mariekers |
15+
16+
As of March 21, help is especially wanted from contributors with experience in Kubernetes and Python.
1817

1918
## Before You Begin
2019

21-
- Join the [#covid19-chime-penn](https://codeforphilly.org/chat?channel=covid19-chime-penn) channel, and [request write access](https://codeforphilly.slack.com/archives/CV4NGQYMP/p1584665484368300) to the `CodeforPhilly/chime` Github repository.
22-
- Our highest-priority work is organized in the [Github Project Management board](https://github.com/CodeForPhilly/chime/projects/2). Look for an issue matching your interests & skills in one of the "Ready" columns.
23-
- Assign yourself to the issue and add a comment to briefly describe your plan.
24-
- For new ideas, please first discuss the change(s) you wish to make via [issue](https://github.com/codeforphilly/chime/issues) or appropriate Slack channel (platform/app - #chime-app, devops - #chime-ops, modeling - #chime-analysis).
20+
- Join the [#covid19-chime-penn](https://codeforphilly.org/chat?channel=covid19-chime-penn) channel for project-wide announcements.
21+
- Our highest-priority work is organized in [Github Project Boards](https://github.com/CodeForPhilly/chime/projects). Look for an issue matching your interests & skills in the "Ready" columns.
22+
- Comment on the issue to let us know you're picking it up, briefly describe your plan, and go forth!
23+
- For new ideas, please first discuss the change(s) you wish to make via [issue](https://github.com/codeforphilly/chime/issues) or the appropriate Slack channel (see table above).
2524

2625
## Making and Submitting Changes
2726

2827
- [Fork](https://help.github.com/en/github/getting-started-with-github/fork-a-repo) the [CodeforPhilly/chime](https://github.com/CodeForPhilly/chime) repo.
2928
- Base your work on the `develop` branch.
3029
- Submit pull requests from your fork, also against the `develop` branch of the `CodeforPhilly/chime` repo.
30+
- Request review from the relevant maintainer(s).
3131
- Check your pull request periodically to see if any changes have been requested or any merge conflicts have arisen.
3232
- If a merge conflict arises, rebase against the latest `develop` branch and force-push the new branch as early as you can. You may need to do this more than once before your changes get merged. Do your best to keep your branch in a mergeable state until it is finished being reviewed and accepted.
3333
- Note: Frequent contributors with write access can submit pull requests from a new branch in the `CodeforPhilly/chime` repository.
@@ -39,8 +39,8 @@ As of March 19, help is especially wanted in these areas:
3939
- Maintainers will review pull requests, asking for changes as needed.
4040
- Once a PR has 1 approval and passes automated tests, maintainers will merge to release branches.
4141
- Exception: Documentation or infrastructure support PRs can be merged directly to master.
42-
- Deploys to production must be signed off on by @beckerfluffle or @cchivers.
42+
- Deploys to production must be signed off by @beckerfluffle or @cchivers.
4343

4444
## Requesting Functionality
4545

46-
**Stakeholders/Users**: Check the [#covid19-chime-penn](https://codeforphilly.org/chat/covid19-chime-penn) channel on [Slack](https://codeforphilly.org/chat) to see if your need is already being addressed.
46+
**Stakeholders/Users**: Check the [#chime-users](https://codeforphilly.org/chat/chime-users) channel on [Slack](https://codeforphilly.org/chat) to see if your need is already being addressed.

docs/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
# CHIME Documentation
1+
# CHIME Developer Documentation
22

3-
This book is the living documentation for developers and operators working on [the CHIME project](https://github.com/CodeForPhilly/chime) or running instances of it.
3+
This book is the living documentation for developers and operators working on [the CHIME project](https://github.com/CodeForPhilly/chime) or running instances of it. Hospitals and public officials, please see [User Documentaiton](https://code-for-philly.gitbook.io/chime/).
44

55
It is maintained in the [GitBook format](https://gitbookio.gitbooks.io/docs-toolchain/structure.html) within [the `docs/` directory](https://github.com/CodeForPhilly/chime/tree/develop/docs) of the project's main Git repository, and new versions are published automatically upon merge to the `develop` branch to: [codeforphilly.github.io/chime](https://codeforphilly.github.io/chime)
66

docs/contributing/app-dev.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ See [Streamlit's Getting Started guide](https://docs.streamlit.io/getting_starte
2424
```bash
2525
pipenv shell
2626
pipenv install
27-
streamlit run app.py
27+
streamlit run src/app.py
2828
```
2929

3030
### With `conda`
@@ -33,7 +33,7 @@ streamlit run app.py
3333
conda env create -f environment.yml
3434
source activate chime
3535
pip install streamlit
36-
streamlit run app.py
36+
streamlit run src/app.py
3737
```
3838

3939
### Choosing a Different Port
@@ -42,15 +42,15 @@ If you need to run the application on a different port than the default (8000),
4242

4343
```bash
4444
export STREAMLIT_SERVER_PORT=1234
45-
streamlit run app.py
45+
streamlit run src/app.py
4646
```
4747

4848
## Project Layout
4949

5050
### Application files
5151

52-
- `app.py`: Main source for the application
53-
- `test_app.py`: [pytest](https://docs.pytest.org/en/latest/) tests for `app.py`
52+
- `src/app.py`: Main source for the application
53+
- `src/test_app.py`: [pytest](https://docs.pytest.org/en/latest/) tests for `app.py`
5454
- `script/`: Developer workflow scripts following [GitHub's Scripts To Rule Them All](https://github.com/github/scripts-to-rule-them-all) pattern.
5555
- `.streamlit/`: [Streamlit config options](https://docs.streamlit.io/cli.html)
5656
- `.env`: Local environment variables to use when running application, this file is copied from `.env.example` to start you out and then ignored by git

0 commit comments

Comments
 (0)