rmlmapper-java & yarrrml-parser mismatch for escaped delimiter

RMLio / rmlmapper-java

The RMLMapper executes RML rules to generate high quality Linked Data from multiple originally (semi-)structured data sources

http://rml.io

MIT License

147 stars 61 forks source link

Closed florent-andre closed 12 months ago

florent-andre commented 2 years ago

rmlmappper-java use not escaped \t : csvw:delimiter "\t"; : rmlmappper test
yarrrml-parser create escaped \t : csvw:delimiter "\\t"; parser test
N3.js library states that this should be an escaped \t : witer test
So it may be a bug in rmlmapper.
- I don't dive more on that (switching from tsv to csv...), but it may catch your attention

DylanVanAssche commented 2 years ago

@pheyvaer Any opinion here?

pheyvaer commented 2 years ago

There is an issue for this in YARRRML parser already, I think.

florent-andre commented 2 years ago

You are right, just found it.

And if I well understand the question you ask to N3.js, Yarrrml-parser is true with escaped \t.

To my understanding, rmlmappper-java use escaped \t and no more not escaped \t.

What do you think ?

DylanVanAssche commented 2 years ago

@florent-andre

What do you think ?

We're waiting for N3.js to fix https://github.com/rdfjs/N3.js/issues/221. Feel free to comment on the upstream issue.

DylanVanAssche commented 12 months ago

Issue was fixed upstream, closing.