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

Tagger parser transfer#9 #28

Closed
wants to merge 4 commits into from
Closed

Conversation

MateuszOlko
Copy link
Contributor

@MateuszOlko MateuszOlko commented Nov 4, 2019

closes: #27
Don't change the text below, it will create checklists after you submit the form :)

Checklist before requesting the review:

  • I checked that all experiments are organized in a dvc pipeline
  • I listed all data & source code depencencies in every dvc file
  • I listed and all outputs in every dvc file
  • I ran dvc commit to check if dvc files reflect the state of my data
  • I successfully ran dvc push before opening this pull request

FYI dvc commit didn't run successfully. There is nkjp2justpos.json file missing. I can fix it and rerun all experiments, but it would take hours. Dvc can't track training properly anyway so i leave it as it is.

Checklist for reviewer:

  • Filenames and function names follow conventions set in README.md
  • Scripts don't have hardcoded settings (global variables at the top of file and default click.option values are ok)
  • After git checkout branch-name and dvc pull, dvc repro related-pipeline-name.dvc loads everything from cache

@MateuszOlko MateuszOlko requested a review from Gizzio November 7, 2019 17:17
@Gizzio Gizzio closed this Oct 7, 2022
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.

Benchmark dependency parser
2 participants