Skip to content
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.

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

Corrections to the requirements.txt and README #7

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

FauzanFarooqui
Copy link

  • Two missing packages were added in the requirements.txt - tqdm and PyYAML.
  • Pandas version is downgraded to the version right before 2.0.0, as the new v2.0 no longer supports df.concat
  • The correct git clone link is updated in the README
  • Example for the evaluation command is corrected to include the config file
  • Some more explanation / example regarding passing the config file while training and the resume option is included.

Added tqdm and PyYAML, with enforced pandas version.
Updated git clone address and added some config help
Corrected evaluation command to include config file
@FauzanFarooqui FauzanFarooqui marked this pull request as ready for review May 25, 2023 12:41
model and data loader
model type, epochs, batch size
relation and entity slot size updated, patience epochs
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.

1 participant