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

Outdated note on partial TD ? #714

Closed
mlagally opened this issue Mar 3, 2022 · 6 comments
Closed

Outdated note on partial TD ? #714

mlagally opened this issue Mar 3, 2022 · 6 comments
Assignees

Comments

@mlagally
Copy link
Contributor

mlagally commented Mar 3, 2022

Please check if the note in the terminology / partial TD definition is still accurate.
From a cursory read of the scripting spec, the produce method requires a fully validated TD.

@egekorkan
Copy link
Contributor

Scripting API says:

Specifically, an ExposedThingInit value is a dictionary used for the initialization of an ExposedThing and it represents a Partial TD as described in the [WOT-ARCHITECTURE]

Thus, a partial TD is used for produce method. A full TD cannot be used for it in most cases since only the runtime knows which ports are available, what is the IP address to be used etc. For MQTT, a full TD might make sense but in any case, a partial TD is given to the produce method

@egekorkan
Copy link
Contributor

The definition in the Architecture spec seems to be correct, propose closing

@danielpeintner
Copy link
Contributor

@egekorkan not sure now...
Do you expect any change in the Scripting API document?

@egekorkan
Copy link
Contributor

Do you expect any change in the Scripting API document?

No I don´t think so, both documents seem fine regarding this definition?

@mlagally
Copy link
Contributor Author

The link in the informative note is wrong.
It points to a published version, which does not mention partial TDs.

@mlagally
Copy link
Contributor Author

Arch call on 10.3.:
Agreed to remove the link.

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

3 participants