Skip to content

Conversation

@kazemaksOG
Copy link

  1. Updated the readme to reflect API changes.
  2. Exposed Docker ports so PostgreSQL is exposed, needed for notebooks.
  3. Updated some python libraries that changed API, added requirements.txt to ease Python env setup.

Copy link
Member

@gillesmag gillesmag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!


In order to update the database you must download all 4 data sources and parse them.

**Upadte**: Semantic scholar API is now behind a license. Ensure your usage fits the license specification.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
**Upadte**: Semantic scholar API is now behind a license. Ensure your usage fits the license specification.
**Update**: Semantic scholar API is now behind a license. Ensure your usage fits the license specification.

```

3. Download the [Open Academic Graph](https://www.aminer.org/open-academic-graph) dataset which contains the Aminer and MAG papers using the following commands:
3. **UPDATE**: aminer and mag datasets now seem to be combined into one graph. Name is kept for scripts to still function. Download the [Open Academic Graph](https://www.aminer.org/open-academic-graph) dataset which contains the Aminer and MAG papers using the following commands:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

URL points to a 404.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants