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

Added the option '--marginal-all' and its description to the usage message of tag command. #17

Merged
merged 1 commit into from
Jan 24, 2016

Conversation

jlerouge
Copy link
Contributor

@jlerouge jlerouge commented Nov 7, 2013

I needed an option to show all marginals for my application, and I found it in the source code. But it's neither documented in the help message nor on the website.

…ssage of tag command

I needed an option to show all marginals for my application, and I found it in the source code. But it's neither documented in the help message nor on the website.
@usptact
Copy link

usptact commented Jan 15, 2016

Hi,

Did you use this information to find the n-best taggings? If not, what information is needed to get those predictions?

Thanks

@tavianator
Copy link
Contributor

@usptact Even all the marginals are not enough to get the n-best taggings. The Viterbi implementation would have to be changed.

@usptact
Copy link

usptact commented Jan 18, 2016

@tavianator Thanks for chiming in. I was in doubt whether the marginals are enough to find the n-best taggings. A quick glance on www showed that one of arguably simplest ways would be to use N-Viterbi, which basicaly stores n forward/backward links. I am pretty poor versed in algorithms, especially in clean C, but I will try to find some code which would be easy to use here.

I see this as an additional option in "tag" mode, say "--nbest" which produces n tab delimited columns with label predictions.

@tavianator
Copy link
Contributor

@usptact That makes sense. Actually at some point I do plan to add n-best support to CRFsuite, just haven't found the time. If you (or anyone) beat me to it that would be great!

chokkan pushed a commit that referenced this pull request Jan 24, 2016
Added the option '--marginal-all' and its description to the usage message of tag command.
@chokkan chokkan merged commit 16f7a36 into chokkan:master Jan 24, 2016
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.

4 participants