Skip to content

add conlleval.pl #202

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

Merged
merged 4 commits into from
Mar 9, 2018
Merged

add conlleval.pl #202

merged 4 commits into from
Mar 9, 2018

Conversation

slefrancois
Copy link
Contributor

The URL for the file conlleval.pl no longer works. This adds the file to the repo, with the custom prints we added for processing the output.

# usage: conlleval [-l] [-r] [-d delimiterTag] [-o oTag] < file
# README: http://www.clips.uantwerpen.be/conll2000/chunking/output.html
# options: l: generate LaTeX output for tables like in
# http://cnts.uia.ac.be/conll2003/ner/example.tex
Copy link
Member

Choose a reason for hiding this comment

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

Maybe over-eager, but that URL does not work and seems to have been replaced by http://www.clips.uantwerpen.be/conll2003/ner/example.tex

# with $boundary fields (default -X-).
# url: http://www.clips.uantwerpen.be/conll2000/chunking/
# started: 1998-09-25
# version: 2004-01-26
Copy link
Member

Choose a reason for hiding this comment

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

Should we update the version number then?

@@ -16,7 +16,7 @@
# with $boundary fields (default -X-).
# url: http://www.clips.uantwerpen.be/conll2000/chunking/
# started: 1998-09-25
# version: 2004-01-26
# version: 2018-03-09
# author: Erik Tjong Kim Sang <erikt@uia.ua.ac.be>
Copy link
Member

Choose a reason for hiding this comment

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

Maybe change author to "original author" to have this more clear. Also indent the "with modification" under "Erik" to have the modification more clear

@nouiz
Copy link
Member

nouiz commented Mar 9, 2018

As @lamblin already approuved, I'll merge.

@nouiz nouiz merged commit 7f06bc2 into lisa-lab:master Mar 9, 2018
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.

3 participants