search

freme-project / freme-ner

Apache License 2.0
6 stars 1 forks source link

Initial labelmatch implementation #173

Closed sandroacoelho closed 7 years ago

fsasaki commented 7 years ago

This change has no issue related to it. @sandroacoelho , can you summarize in an issue what has been done? Also, one needs to update the documentation, I assume, to have the new parameter labelmatch documented? Putting @jnehring into the loop FYI.

fsasaki commented 7 years ago

Thanks for referencing https://github.com/freme-project/freme-ner/issues/171 , Sandro. That issue shows the motivation of the change. It would be nice to also have the implementation briefly (bullet points are enough!) summarised and the documentation updated.

sandroacoelho commented 7 years ago

Hi @fsasaki. As you have mentioned, this issue is related with our Label Matching approach and contains

It is a very beginning implementation. We need to test the approach before document it (cc: @m1ci, @gustavo).

I will keep you updated.

Best,

fsasaki commented 7 years ago

Makes sense, thanks, Sandro.

2017-01-23 11:30 GMT+01:00 Sandro notifications@github.com:

Hi @fsasaki https://github.com/fsasaki. As you have mentioned, this issue is related with our Label Matching approach and contains

  • A new endpoint to deal with Label Matching: /e-entity/freme-ner/labelmatch
  • A spotter(string matching algorithm) + dictionary;

It is a very beginning implementation. We need to test the approach before document it (cc: @m1ci https://github.com/m1ci, @gustavo https://github.com/gustavo).

I will keep you updated.

Best,

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/freme-project/freme-ner/pull/173#issuecomment-274451529, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH5AsC7u-csBSWn7u_m1YL5mOIPK7IRks5rVIEzgaJpZM4LqdqE .