Skip to content

Source NYT #259

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

Merged
Kilo59 merged 13 commits into from
Apr 13, 2020
Merged

Source NYT #259

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
8751f7e
Merge pull request #1 from ExpDev07/master
ibhuiyan17 Apr 3, 2020
e8cd213
Started adding documentation for NYT source
ibhuiyan17 Apr 3, 2020
9ffe9e6
Started nyt.py file in services.location module
ibhuiyan17 Apr 7, 2020
4903abb
Testing nischal pushing to branch
nischalshankar Apr 8, 2020
6a744a6
Merge pull request #2 from ibhuiyan17/source-nyt
ibhuiyan17 Apr 8, 2020
a9c5f37
Merge remote-tracking branch 'upstream/master'
nischalshankar Apr 8, 2020
736b90d
Merge branch 'master' of https://github.com/ibhuiyan17/coronavirus-tr…
nischalshankar Apr 8, 2020
ce841d3
Temp directory to test parsing nyt timeseries
ibhuiyan17 Apr 10, 2020
5f587a9
Locally working nyt source on v2
nischalshankar Apr 10, 2020
cf9694c
Deleted temporary folder so it hopefully passes build
ibhuiyan17 Apr 10, 2020
8a27080
Added mock data, unit tests, and regression tests for source NYT
ibhuiyan17 Apr 12, 2020
59437b3
Updated README to reflect new NYT source
nischalshankar Apr 12, 2020
149de08
Fixed requested points from PR
ibhuiyan17 Apr 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Updated README to reflect new NYT source
  • Loading branch information
@nischalshankar
nischalshankar committed Apr 12, 2020
commit 59437b3fc9305cb604bebf285dd99a885376455b
8 changes: 6 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,13 +24,17 @@ Support multiple data-sources.
![Covid-19 Recovered](https://covid19-badges.herokuapp.com/recovered/latest)
![Covid-19 Deaths](https://covid19-badges.herokuapp.com/deaths/latest)

## New York Times is now available as a source!

**Specify source parameter with ?source=nyt. NYT also provides a timeseries! To view timelines of cases by US counties use ?source=nyt&timelines=1**
Comment thread
ibhuiyan17 marked this conversation as resolved.
Outdated

## Recovered cases showing 0

**JHU (our main data provider) [no longer provides data for amount of recoveries](https://github.com/CSSEGISandData/COVID-19/issues/1250), and as a result, the API will be showing 0 for this statistic. Apolegies for any inconvenience. Hopefully we'll be able to find an alternative data-source that offers this.**
**JHU (our main data provider) [no longer provides data for amount of recoveries](https://github.com/CSSEGISandData/COVID-19/issues/1250), and as a result, the API will be showing 0 for this statistic. Apologies for any inconvenience. Hopefully we'll be able to find an alternative data-source that offers this.**

## Available data-sources:

Currently 2 different data-sources are available to retrieve the data:
Currently 3 different data-sources are available to retrieve the data:

* **jhu** - https://github.com/CSSEGISandData/COVID-19 - Worldwide Data repository operated by the Johns Hopkins University Center for Systems Science and Engineering (JHU CSSE).

Expand Down