Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Add list of pretrained models to README #246

Merged
merged 3 commits into from
Apr 7, 2021
Merged

Conversation

epwalsh
Copy link
Member

@epwalsh epwalsh commented Apr 7, 2021

Also adds a script and a GitHub Actions job for keeping this list up-to-date. The script can be ran to update the list. The Actions job will fail if the script needs to be ran.

@@ -8,7 +8,7 @@
"archive_file": null,
"training_config": null,
"overrides": null,
"install_instructions": "pip install allennlp==1.0.0 allennlp-models==1.0.0",
"install_instructions": "pip install allennlp==1.0.0 allennlp-models==1.0.0"
Copy link
Member

Choose a reason for hiding this comment

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

The template should probably say 2.2.0 by now.

Copy link
Member

@dirkgr dirkgr left a comment

Choose a reason for hiding this comment

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

This reminds me, all the people working on models right now need to add a model card as well. Otherwise their work will disappear into obscurity!

@epwalsh epwalsh merged commit b358dad into main Apr 7, 2021
@epwalsh epwalsh deleted the more-readme-updates branch April 7, 2021 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants