Skip to content
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

arch-id-correlation : An identifier in the WoT Thing Description MUST allow for the correlation of multiple TDs representing the same original Thing or ultimately unique physical entity. #635

Closed
Tracked by #625
mlagally opened this issue Nov 17, 2021 · 9 comments

Comments

@mlagally
Copy link
Contributor

mlagally commented Nov 17, 2021

Please follow the process in #641

@benfrancis
Copy link
Member

benfrancis commented Nov 18, 2021

From #625 (comment)...

  • arch-id-correlation : An identifier in the WoT Thing Description MUST allow for the correlation of multiple TDs representing the same original Thing or ultimately unique physical entity.

I personally don't really agree with this assertion because in my opinion, as far as a WoT Consumer is concerned, two different Thing Descriptions hosted at two different URLs for the same physical device should be treated as two separate Web Things.

However, the Thing Description specification does define an id member of the Thing Description and there's nothing to stop two Thing Descriptions using the same id. It's currently not exactly clear what a Consumer should do in this situation, but let's continue that discussion in w3c/wot-discovery#190.

I do think the term "correlation" in this assertion is a bit ambiguous.

Conclusion: ❌ I don't really agree with this assertion, but let's continue the discussion.


Suggested action: Remove the assertion.

@k-toumura
Copy link
Contributor

It we make it mandatory to be able to correlate IDs, there is a concern about privacy issues.
For use cases where the Intermediary is hiding the Things behind it, it may be worthwhile not to be able to correlate IDs.

@sebastiankb
Copy link
Contributor

I think, the task of the TD is to provide the option for identification when required.

@k-toumura

For use cases where the Intermediary is hiding the Things behind it, it may be worthwhile not to be able to correlate IDs.

Yes and I think this is the task of an ecosystem which manages the Things and provides potential IDs. This should not be specified by a standard.

@sebastiankb
Copy link
Contributor

sebastiankb commented Nov 24, 2021

from today's TD call

@mlagally
Copy link
Contributor Author

mlagally commented Nov 25, 2021

Arch call on 25:11:
Need to improve text, discuss in TD, discovery
Remove assertion from arch.

@mlagally
Copy link
Contributor Author

mlagally commented Dec 16, 2021

Arch call on 16.12:
Did the discussion in TD and Discovery group happen?
Need to ask @mmccool about Discovery,
TD discussion seems to be still open

@mlagally
Copy link
Contributor Author

Arch call on 19.5.:
@mmccool Please remove the assertion.

@farshidtz
Copy link
Member

In the past, we've discussed this in length in w3c/wot-discovery#190.

IMO, since Thing.id is the @id field of TD, we should respect the following JSON-LD spec:

@id
Used to uniquely identify node objects that are being described in the document with IRIs or blank node identifiers.

The only way to let @id uniquely identify the node object (TD) is to consider id as the unique identifier of TD!

@mlagally
Copy link
Contributor Author

@farshidtz
We implemented the change requested by this issue. Further discussions about ids should be further continued in the TD-TF, if necesary.

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

No branches or pull requests

6 participants