freme-project / e-Entity

Apache License 2.0
1 stars 1 forks source link

P3: Filter NIF log messages #9

Closed m1ci closed 9 years ago

m1ci commented 9 years ago

There are log messages in the NIF output. I believe most people dont care about these log messages so they generate noise. My suggestion: We create a log filter in ConversionServices that filters out NIF log messages from a Jena model. This filter is enabled by default. We create an optional GET parameter “logging=true” that disables the filter.

This is e-Entity related feature, and I do not see reason this to be part of the ConversionServices. BTW, log messages are good for debugging, also there provide very useful metadata information.

<urn:uuid:1eb52471-ce7e-4761-a161-07fb201be3ba>
    a             rlog:Entry ;
    rlog:date     "2015-04-27T13:09:22.072Z"^^xsd:dateTime ;
    rlog:level    rlog:DEBUG ;
    rlog:message  "Finished creating 1 sentence(s) with 4 word(s) " ;
    dc:creator    "org.nlp2rdf.implementation.spotlight.SpotlightWrapper" .

<urn:uuid:0ef8de6c-9517-47bd-a6eb-c20f202b006c>
    a             rlog:Entry ;
    rlog:date     "2015-04-27T13:09:22.292Z"^^xsd:dateTime ;
    rlog:level    rlog:DEBUG ;
    rlog:message  "Annotated 1 nif:Context(s)  in 224.0 ms. (avg 224.0) producing 71 triples" ;
    dc:creator    "org.nlp2rdf.implementation.spotlight.SpotlightWS" .

I wouldn't call this noise. But, yes, we can implement such logging parameter. Let's see what other say.

jnehring commented 9 years ago

We can introduce this functionality in the second prototype.

m1ci commented 9 years ago

no log messages in the FREME NER. The log are part of the DBpedia Spotlight implementation which we don't maintain.

m1ci commented 9 years ago

fixed, log messages are filtered out. The changes will be deployed in the following hours.

m1ci commented 9 years ago

no more log messages in the output. See http://api-dev.freme-project.eu/current/e-entity/dbpedia-spotlight/documents/?informat=text&input=Berlin+is+in+Germany&outformat=turtle

If everything is OK, please close the issue. Thanks!