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

DEX-00 comments #137

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

DEX-00 comments #137

wants to merge 7 commits into from

Conversation

barakgafni
Copy link
Collaborator

Edits to the draft, including updates regarding the role of each type of node in the network, generalization of exporting the data from the network, and further discussions on performance, security and hop limit

Change the overview as we agreed that the export is not always for a collector
but rather from the network in general
there was misalignment between the fixed paragraph, which state that all must export data, and the following which say the may. I think that MAY is the right option, so it was fixed.
data may not be sent out of the device, hence the text should reflect that
@barakgafni
Copy link
Collaborator Author

One more question to consider is regarding the checksum complement bit - whether it MUST be zero and whether nodes along the way should check it. I believe that it should just be ignored is stated in the draft already, but need to decide on the requirement from the encapsulator.

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.

1 participant