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.

Sign up for GitHub

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

Jump to bottom

Update protobuf documentation to reflect that metric connections are supported by the aggregation endpoint #62

Closed
stefan-brus-frequenz opened this issue Nov 25, 2024 · 0 comments · Fixed by #80
Closed

Update protobuf documentation to reflect that metric connections are supported by the aggregation endpoint #62

stefan-brus-frequenz opened this issue Nov 25, 2024 · 0 comments · Fixed by #80
Labels
part:docs Affects the documentation part:proto Affects the protocol buffer definition files priority:high Address this as soon as possible type:enhancement New feature or enhancement visitble to users
Milestone
v1.0.0

Comments

@stefan-brus-frequenz
Copy link
Collaborator

What's needed?

The documentation strings related to how formulas are handled by the aggregation endpoint should reflect that metric connections are supported, in the following way: #COMPONENT_ID[CONNECTION_STRING]

Example: #3[connection0] + #2[connection0]

Proposed solution

No response

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response
@stefan-brus-frequenz stefan-brus-frequenz added part:docs Affects the documentation part:proto Affects the protocol buffer definition files priority:high Address this as soon as possible type:enhancement New feature or enhancement visitble to users labels Nov 25, 2024
@stefan-brus-frequenz stefan-brus-frequenz added this to the v1.0.0 milestone Nov 25, 2024
stefan-brus-frequenz added a commit to stefan-brus-frequenz/frequenz-api-reporting that referenced this issue Feb 3, 2025
@stefan-brus-frequenz
Show support for metric connections in aggregated formula docs
5336bec 
Fixes frequenz-floss#62
@stefan-brus-frequenz stefan-brus-frequenz mentioned this issue Feb 3, 2025
Merged
stefan-brus-frequenz added a commit to stefan-brus-frequenz/frequenz-api-reporting that referenced this issue Feb 3, 2025
@stefan-brus-frequenz
Show support for metric connections in aggregated formula docs
43ccd9e 
Fixes frequenz-floss#62

Signed-off-by: Stefan Brus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
part:docs Affects the documentation part:proto Affects the protocol buffer definition files priority:high Address this as soon as possible type:enhancement New feature or enhancement visitble to users
Projects
None yet
Milestone
v1.0.0
1 participant
@stefan-brus-frequenz