Skip to content

add conlleval.pl#202

Merged
nouiz merged 4 commits intolisa-lab:masterfrom
slefrancois:conlleval
Mar 9, 2018
Merged

add conlleval.pl#202
nouiz merged 4 commits intolisa-lab:masterfrom
slefrancois:conlleval

Conversation

@slefrancois
Copy link
Copy Markdown
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.

Comment thread code/conlleval.pl Outdated
# 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
Copy Markdown
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

Comment thread code/conlleval.pl Outdated
# with $boundary fields (default -X-).
# url: http://www.clips.uantwerpen.be/conll2000/chunking/
# started: 1998-09-25
# version: 2004-01-26
Copy link
Copy Markdown
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?

Comment thread code/conlleval.pl Outdated
# started: 1998-09-25
# version: 2004-01-26
# version: 2018-03-09
# author: Erik Tjong Kim Sang <erikt@uia.ua.ac.be>
Copy link
Copy Markdown
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
Copy Markdown
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