Skip to content
This repository was archived by the owner on Jan 1, 2024. It is now read-only.

Some fix in dataset construction#11

Open
andreabac3 wants to merge 3 commits intofacebookresearch:mainfrom
andreabac3:main
Open

Some fix in dataset construction#11
andreabac3 wants to merge 3 commits intofacebookresearch:mainfrom
andreabac3:main

Conversation

@andreabac3
Copy link

Hi,

I am working on NeuralDB, and I want to contribute to the original project with these small fixes.

  • I have improved the requirements.txt since some packages are missing
  • I have added a setup.sh in order to install the requirements and also to install the NLTK punk dependency.
  • I have also removed the logger in the file make_database_initial.py, since ndb_data.util.log_helper is missing.

Kind regards,
Andrea Bacciu

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants