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

Terminology review with IETF #196

Merged
merged 1 commit into from
Apr 5, 2019
Merged

Conversation

mkovatsc
Copy link
Contributor

@mkovatsc mkovatsc commented Apr 3, 2019

We had a workshop with IETF people to check the compatibility of the definitions around affordances, hypermedia controls, etc.

We worked out a good model, but noticed that the definitions of many terms had drifted toward wrong statements.

This PR provides corrections and removes some terminology entries that are unneeded (e.g., terms that are never used in the document).

There needs to be another PR that updates the longer definitions under Section 6. We also made some action items to align properly in the TD document.

Another identified issue is the new text on "Wot Servients" in Section 6.1, the use of "WoT Client/Server role", and the confusion of Servient and intermediary.

@mkovatsc mkovatsc requested a review from mlagally April 3, 2019 16:14
@mlagally
Copy link
Contributor

mlagally commented Apr 4, 2019

@mkovatsc thanks very much for the review. Are there meeting notes of the IETF workshop?

@mkovatsc
Copy link
Contributor Author

mkovatsc commented Apr 4, 2019

@takuki
Copy link
Contributor

takuki commented Apr 4, 2019

Another identified issue is the new text on "Wot Servients" in Section 6.1, the use of "WoT Client/Server role", and the confusion of Servient and intermediary.

Sorry that I could not attend the architecture telecon. What kind of issues and confusions were found with regards to section 6.1?

@k-toumura
Copy link
Contributor

In the part of 4th April architecture telecon, we discussed about unclear usage of terminology client/servient/server, application/gateway/proxy/twin/device, etc.
After discussion, we've decided not to use the term client/server role in Section 4.3 (Use case summary) because it impose client role (actively initiate a communication) or server role (passively wait a communication) to devices, applications, edge devices.

And also, @mkovatsc proposes to update terminology and some part of this PR is related with above discussion ('Servient' is a software stack, not a role).

Due to limited time for CR submission, @mlagally plans to merge this PR after today's TD call
(Friday at 4am US Eastern / 1am US Pacific / 10am Europe / 5pm Japan) if there is no objection.
And based on this result, @mkovatsc will start update Section 6.
@takuki , please review the PR (and issue #195, #146, #188) and comment if you have an opinion for this.

@takuki
Copy link
Contributor

takuki commented Apr 5, 2019

Thank you @k-toumura -san. I left a comment in #188.

index.html Show resolved Hide resolved
also used to refer to a Servient in server role only.</dd>
<dt>
<dfn>WoT Servient</dfn>
</dt>
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be pragmatic to have both terms WoT Servient and Servient defined in the document.

Copy link
Contributor

@mlagally mlagally left a comment

Choose a reason for hiding this comment

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

Discussed and approved in the call on 5.4.

@mlagally mlagally merged commit 2bf3484 into master Apr 5, 2019
@mkovatsc mkovatsc deleted the mkovatsc-ietf-terminology-review branch April 12, 2019 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants