Skip to content

Release v4.0 #166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 34 commits into from
Sep 3, 2021
Merged

Release v4.0 #166

merged 34 commits into from
Sep 3, 2021

Conversation

kateharwood
Copy link
Contributor

@kateharwood kateharwood commented Sep 3, 2021 •
edited
Loading

  • Addition of "as of" feature for observed values plot
  • Addition of the option to include forecasts themselves in the observed value plot
  • Additions to README
  • Add <title> to page
  • Minor bug fixes

nmdefries and others added 30 commits July 29, 2021 16:13
@nmdefries
Merge pull request #153 from cmu-delphi/main
2b23d49 
Cleanup: merge main into dev
@kateharwood
first draft of as of
6f1d4f3
the rest of the as-of implementation
b3bb511
@kateharwood
Merge branch 'dev' into as-of
4249280
factoring out function and removing dup fix
ed5f5b2
@nmdefries
change working dir to action user so files can be deleted
d68654b 
Some data files from a previous run of this action are created with root
as the owner. Because the action runs as a non-root user, such files are
[not able to be deleted](https://github.com/cmu-delphi/forecast-eval/runs/3351290227?check_suite_focus=true)
which is needed for the actions/checkout@v2 step
to install the newest version of the repo.

This seems to be a bug in the actions/checkout@v2 command, perhaps
specific to self-hosted workflows. [Summary of
issue](actions/runner#691) and [suggested
fixes[(actions/checkout#211).
@nmdefries
Merge pull request #155 from cmu-delphi/fix-permissions-on-action-setup
5178819 
Change repo dir owner to action user so files can be deleted
add option to show forecasts
a581890
changes based on comments
c09d35a
fixing order of min as of choice
b2e9ba1
@kateharwood
Merge pull request #154 from cmu-delphi/as-of
48f63ad 
"As of" feature
Merge branch 'dev' into add-forecasts
a5a0a4a
fixing merge conflicts
33a1fcc
moving meat to its own function
151df9e
adding stringr library
28ce03e
@kateharwood
Merge pull request #156 from cmu-delphi/add-forecasts
e32820c 
Add forecasts
bug fixes for as of and forecaster features
962e1c9
fix nation as of date for deaths
5d5c364
adding forecasts to total locations plot
b21690d
@kateharwood
Merge pull request #158 from cmu-delphi/as-of-forecaster-bug-fixes
7196fe7 
Bug fixes for as of and forecaster features
@kateharwood
Merge branch 'dev' into total-forecasts
0986f70
cleaning up merge conflicts
c687ae5
@kateharwood
Update README.md
47ce269
@kateharwood
Update README.md
948cbd3
@kateharwood
Update README.md
a5a3703
@kateharwood
Merge pull request #159 from cmu-delphi/total-forecasts
7178977 
Adding forecasts to total locations observed as of plot
@kateharwood
Update README.md
97e36a1
add title to page
0e9e5d2
fix as of current date
56807a2
@kateharwood
Merge pull request #162 from cmu-delphi/add-page-title
5f5bdf6 
Add title to page
@kateharwood kateharwood merged commit cfbccbf into main Sep 3, 2021
@nmdefries nmdefries deleted the release_v4.0 branch September 15, 2021 15:08
@nmdefries nmdefries mentioned this pull request Sep 23, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nmdefries nmdefries nmdefries approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kateharwood @nmdefries