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

Add parameter-dataset.tsv sample links in README.md #29

Merged
merged 4 commits into from
Apr 17, 2019

Conversation

johnnychhsu
Copy link
Contributor

@johnnychhsu johnnychhsu commented Apr 3, 2019

When trying to train a model, I have met some problems, that can be explained in README.md which will help future users and save their time.

  1. Add parameter-dataset.tsv sample links in README.md
  2. Add GENIE_PARSER_PATH setting in README.md
  3. Fix split-train-eval command, remove -i and []

README.md Outdated Show resolved Hide resolved
@johnnychhsu
Copy link
Contributor Author

Fix it, thanks!

@johnnychhsu johnnychhsu merged commit bd040df into master Apr 17, 2019
@johnnychhsu johnnychhsu deleted the param_dataset_example branch April 17, 2019 09:02
gcampax added a commit that referenced this pull request Jun 23, 2020
It's a required capability.

This commit is mostly because at some point we had at least one
platform (iirc, almond-cloud) with enabled permissions but with
no smt solver available. With this, we would fail loudly enough
to catch the problem when bootstrapping the platform.

Fixes #29
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