Skip to content

Commit

Permalink
Remove ComponentBoundsTargetMetric
Browse files Browse the repository at this point in the history
This enum has been removed in favour of the `Metric` enum from
`frequenz-api-common`.

Signed-off-by: Tiyash Basu <tiyash.basu@frequenz.com>
  • Loading branch information
tiyash-basu-frequenz committed Mar 27, 2024
1 parent fb4db18 commit 76ff6c9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 13 deletions.
3 changes: 3 additions & 0 deletions RELEASE_NOTES.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
inclusive bounds. The old RPCs `AddComponentInclusionBounds` and
`AddComponentExclusionBounds` have been removed.

- The enum `ComponentBoundsTargetMetric` has been removed in favour of the
`Metric` enum from `frequenz-api-common`.

<!-- Here goes notes on how to upgrade from previous versions, including deprecations and what they should be replaced with -->

## New Features
Expand Down
14 changes: 1 addition & 13 deletions proto/frequenz/api/microgrid/v1/microgrid.proto
Original file line number Diff line number Diff line change
Expand Up @@ -439,25 +439,13 @@ message ReceiveSensorDataStreamResponse {
frequenz.api.common.v1.microgrid.sensors.SensorData data = 1;
}

// An enumerated list of metrics whose bounds can be set using the RPCs
// `AddComponentExclusionBounds` and `AddComponentInclusionBounds`.
enum ComponentBoundsTargetMetric {
COMPONENT_BOUNDS_TARGET_METRIC_UNSPECIFIED = 0;
COMPONENT_BOUNDS_TARGET_METRIC_POWER_ACTIVE = 1;
COMPONENT_BOUNDS_TARGET_METRIC_CURRENT = 2;
COMPONENT_BOUNDS_TARGET_METRIC_CURRENT_PHASE_1 = 3;
COMPONENT_BOUNDS_TARGET_METRIC_CURRENT_PHASE_2 = 4;
COMPONENT_BOUNDS_TARGET_METRIC_CURRENT_PHASE_3 = 5;
COMPONENT_BOUNDS_TARGET_METRIC_POWER_REACTIVE = 6;
}

// Request parameters for the RPC `AddComponentBounds`.
message AddComponentBoundsRequest {
// The ID of the target component.
uint64 component_id = 1;

// The target metric whose bounds have to be set.
ComponentBoundsTargetMetric target_metric = 2;
frequenz.api.common.v1.metrics.Metric target_metric = 2;

// A list of bounds for the target metric.
// While incorporating these bounds, the API will ensure that overlapping
Expand Down

0 comments on commit 76ff6c9

Please sign in to comment.