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

Protocol update effort (tracking issue) #2

Closed
12 tasks done
kjetilkjeka opened this issue Jul 17, 2018 · 7 comments
Closed
12 tasks done

Protocol update effort (tracking issue) #2

kjetilkjeka opened this issue Jul 17, 2018 · 7 comments

Comments

@kjetilkjeka
Copy link
Contributor

kjetilkjeka commented Jul 17, 2018

This is an attempt of gathering references to everything concerned by the new protocol. The list will be edited to be kept updated. To keep this issue helpfull instead of just becoming yet another place to discuss things, please keep specialized discussion in the relevant issue.

Administriva

Core protocol

DSDL

@kjetilkjeka
Copy link
Contributor Author

@pavel-kirienko feel free to add/edit stuff, or uncheck things if i were to quick to check them.

@pavel-kirienko
Copy link
Member

I marked the TAO entry as decided since the consensus seems to be in favor of its complete removal. Bye bye TAO.

@kjetilkjeka
Copy link
Contributor Author

kjetilkjeka commented Aug 3, 2018

Settle on whether CRC should include data type signature (or other forms of compatibility analyzis).

I think we've practically reached consensus on that the CRC should not include data type signatures or similar. Check off your ticky box if you agree to the conclusion and that the discussion has been left open long enough. Or edit and add another ticky box if you want to bring the attention to someone else.

@pavel-kirienko
Copy link
Member

It has been suggested to use a bit in the transfer ID, but the current discussion is leaning towards not conveying protocol version info over the wire.

I think the discussion is leaning towards using the toggle bit for that. I took the liberty to reflect that in the draft (sec 4.2.3.2), believing that that was the consensus:

Transfers where the initial value of the toggle bit is zero must be ignored. The initial state of the toggle bit may be inverted in the future revisions of the protocol to facilitate automatic protocol version detection.

@kjetilkjeka
Copy link
Contributor Author

Yes you're absolutely right. Thanks for pointing this out, I've updated the overview now.

@pavel-kirienko
Copy link
Member

Marked the following as resolved:

  • Settle on how the protocol version should be conveyed over the wire
  • Figure out how we handle deprecations of definitions.
  • Figure out if we should do anything to namespacing of dsdl

@pavel-kirienko
Copy link
Member

All important things have been settled upon and resolved, so I am closing this. For future reference, many important decisions have been discussed there: https://forum.uavcan.org/c/dev

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

No branches or pull requests

2 participants