search

kg-construct / rml-cc

RML-CC: Collections and Containers definitions for RML
https://w3id.org/rml/cc/spec
Creative Commons Attribution 4.0 International
3 stars 3 forks source link

Fixing namespaces. Changing `rr:` into `rml:` where necessary. #39

Closed chrdebru closed 1 year ago

frmichel commented 1 year ago

Thx for this @chrdebru. Looks good to me. Only doubt, there remains the rr: namespace definition in overview.md, should we remove it or do we refer to it somewhere?

DylanVanAssche commented 1 year ago

Normally, the namespace can be fully replaced by the RML one

chrdebru commented 1 year ago

I kept the one rr namespace declaration in the example as there is a section on rr:column. Rr:column is not declared in rml core.Sent from my iPhoneOn 7 Jul 2023, at 12:55, Dylan Van Assche @.***> wrote: Normally, the namespace can be fully replaced by the RML one

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

DylanVanAssche commented 1 year ago

Rr:column is not declared in rml core

rr:column is superseded by rml:reference and won't be supported anymore by Core.

chrdebru commented 1 year ago

Yes. But we discussed backwards compatible w.r.t. columns in our spec. We can remove the rr namespace declaration, but then it will need to be mentioned in that section.Sent from my iPhoneOn 7 Jul 2023, at 15:17, Dylan Van Assche @.***> wrote:

Rr:column is not declared in rml core

rr:column is superseded by rml:reference and won't be supported anymore by Core.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

DylanVanAssche commented 1 year ago

Ah but backwards compatibility is already handled in Core for this. But @anaigmo, @andimou and @pmaria know better about this part.

chrdebru commented 1 year ago

I’ll remove it, then!On 7 Jul 2023, at 15:23, Dylan Van Assche @.***> wrote: Ah but backwards compatibility is already handled in Core for this. But @anaigmo, @andimou and @pmaria know better about this part.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>