Skip to content

Conversation

maximelefrancois86
Copy link

This is needed for the ongoing W3C SSN specwork.

The Smart Applications REFerence (SAREF) suite of ontologies is published as a set of open standards produced by ETSI TC SmartM2M. Some references to recent technical specifications were missing or outdated.

In general, the ETSI references are mostly outdated and it would be worth re-generating file etsi.json

@dr-shorthair
Copy link
Contributor

@tidoust are you managing merges (or is it August in France :-) ?)

@tidoust
Copy link
Collaborator

tidoust commented Aug 18, 2025

In general, the ETSI references are mostly outdated and it would be worth re-generating file etsi.json

Yes, see #846, ETSI entries have not been updated for some time already.

The PR changes all lines in etsi.json. I suspect that's probably the result of some re-formatting. Would it be possible to restrict the updates to the actual updates to ease review?

@maximelefrancois86
Copy link
Author

my bad. I added CRLF line endings.
It's all in order now.

Copy link
Collaborator

@tidoust tidoust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I note that all but two entries in the etsi.json file use an http scheme instead of https, whereas HTTP requests to www.etsi.org seem to always return a permanent redirect to the HTTPS URL. It would be great to use https URLs throughout. That's orthogonal to this update though.

"title": "SmartM2M; SAREF reference ontology patterns",
"status": "Published",
"publisher": "ETSI",
"date": "2024"
"date": "2024-01"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A mere 2024 works for a human readable date, but 2024-01 does not. rawDate should be used instead:

Suggested change
"date": "2024-01"
"rawDate": "2024-01"

Copy link
Collaborator

@tidoust tidoust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline for required change to fix validation against schema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants