-
Notifications
You must be signed in to change notification settings - Fork 7
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
XLSX source is not supported #79
Comments
Hi José Antonio, Your mapping is missing the referenceFormulation, I understand that is CSV right?(https://rml.io/yarrrml/spec/#reference-formulation)
|
Hi @neobernad, is this what you expect from the output?
|
Hi @dachafra, Firstly, thanks for the quick response :-). I forgot to mention that I removed the 'referenceFormulation' which formerly was The output you suggest it is something I would expect! Shall I transform any Excel file into a CSV then? |
If you want to use Excel referenceFormulation you need to use its engine and extension: https://www.dfki.uni-kl.de/~mschroeder/demo/excel-rml/. But I guess you want to use an XLSX file without transforming it to CSV but the behavior would be the same, i.e. the expected parsing of the file is per row similar as it would be a CSV (that is what the referenceFormulation means). If this output is what you expect, it's already solved and I'll push the changes |
Now, XLSX is supported with CSV referenceFormulation. |
Sorry for the late response, I could not on this topic until now. I have tested it with the latest version of the repository and with CSV referenceFormulation. It works like a charm, thank you! :-) |
Hi @dachafra,
I have the following YARRRML file (actually it is longer and I have anonymized it):
The execution of the translation of the file above raises an exception
ERROR: The YARRRML mapping has not been translated
:Here,
rml_content
isNone
.What could be wrong? After having a quick look debugging yatter, it seems that the YARRRML file is being properly opened and parsed, but something in the
translate
orget_non_asserted_mappings
methods is not liking the structure of the mappings perhaps?Thanks,
José Antonio
The text was updated successfully, but these errors were encountered: