Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Review the current v0.1 port introspection interface per discussion on the forum #84

Closed
pavel-kirienko opened this issue Oct 4, 2020 · 0 comments · Fixed by #93
Closed

Comments

@pavel-kirienko
Copy link
Member

See https://forum.uavcan.org/t/alternative-transport-protocols-in-uavcan/324/34?u=pavel.kirienko

This affects uavcan.node.port.* (currently v0.1)

@pavel-kirienko pavel-kirienko added this to the 1.0 milestone Oct 4, 2020
@pavel-kirienko pavel-kirienko self-assigned this Oct 4, 2020
pavel-kirienko added a commit that referenced this issue Oct 25, 2020
The reviewed design will address the same demands using a stateless subject and the register interface:

- #84
- #89

The objective is to comply with the design guidelines and to reduce the implementation effort.
pavel-kirienko added a commit that referenced this issue Oct 25, 2020
The new service is stateless and compliant with the interface design guidelines.

The Statistics service is removed and not yet replaced because it had a design issue (did not distinguish between servers and clients) and because YAGNI. Later we may address this using the Register API.

The GetInfo service is removed because for fixed ports the data type information is deducible from the port-ID, and non-fixed ports are addressed by #92
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
1 participant