Skip to content

Name single phase and 3-phase metrics consistently #880

@daniel-zullo-frequenz

Description

@daniel-zullo-frequenz

What's needed?

The name for the metrics streamers are not consistent in term of single phase or 3-phase values that they stream.

microgrid.grid().power.new_receiver() # single phase values

microgrid.grid().current.new_receiver() # 3-phase tuple

microgrid.voltage().new_receiver() # 3-phase tuple
 
microgrid.frequency.new_receiver() # single phase values

The SDK will be adding more 3-phase metrics in the near future so it'll be helpful to distinguish the type of values provided by each metric in the name.

Proposed solution

A first idea is to append a suffix to the 3-phase metrics (e.g. _per_phase). This still needs to be discussed and agreed on.

microgrid.grid().power.new_receiver() # single phase values

microgrid.grid().current_per_phase.new_receiver() # 3-phase tuple

microgrid.voltage_per_phase().new_receiver() # 3-phase tuple
 
microgrid.frequency.new_receiver() # single phase values

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response

Metadata

Metadata

Assignees

Labels

part:data-pipelineAffects the data pipelinepriority:highAddress this as soon as possiblescope:breaking-changeBreaking change, users will need to update their codetype:enhancementNew feature or enhancement visitble to users

Type

No type

Projects

Status

Done

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions