You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The enum with the oder variants was compiled into the following rust enum
(by prost):
pubenumEvChargerType{/// Default type.EvchargerTypeUnspecified = 0,/// The EV charging station supports AC charging only.EvchargerTypeAc = 1,/// The EV charging station supports DC charging only.EvchargerTypeDc = 2,/// The EV charging station supports both AC and DC.EvchargerTypeHybrid = 3,}
Here the enum variants were unnecessarily prefixed with EvchargerType.
This lead to accessing the enum variants in a very verbose manner, e.g., EvChargerType::EvchargerTypeHybrid.
The changed version of the enum in this commit results in the following
rust enum:
pubenumEvChargerType{/// Default type.Unspecified = 0,/// The EV charging station supports AC charging only.Ac = 1,/// The EV charging station supports DC charging only.Dc = 2,/// The EV charging station supports both AC and DC.Hybrid = 3,}
Here the unnecessary prefix EvchargerType is absent. This reduces the
verbosity while accessing the enum variants, making the resulting rust code
more readable, e.g., as EvChargerType::Hybrid.
This change also leads to renaming the enum from EVChargerType to EvChargerType, to satisfy protolint requirements.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Frequenz Common API Release Notes
Summary
This release includes renaming the
EVChargerType
enum toEvChargerType
andthe addition of the
MetricAggregation
message.Upgrading
The submodule URL was changed to use HTTPS instead of SSH (to avoid problems trying to unlock SSH keys to do updates, etc.).
Make sure you sync your submodules to the new URL:
EVChargerType
enum refactoredThe enum with the oder variants was compiled into the following rust enum
(by prost):
Here the enum variants were unnecessarily prefixed with
EvchargerType
.This lead to accessing the enum variants in a very verbose manner, e.g.,
EvChargerType::EvchargerTypeHybrid
.The changed version of the enum in this commit results in the following
rust enum:
Here the unnecessary prefix
EvchargerType
is absent. This reduces theverbosity while accessing the enum variants, making the resulting rust code
more readable, e.g., as
EvChargerType::Hybrid
.This change also leads to renaming the enum from
EVChargerType
toEvChargerType
, to satisfy protolint requirements.New Features
Added
MetricAggregation
messageThe message
MetricAggregation
has been copied from frequenz-api-microgridinto this repository, so that it can be used by other APIs as well.
Bug Fixes
None
What's Changed
EVChargerType
enum variants by @tiyash-basu-frequenz in RefactorEVChargerType
enum variants #21MetricAggregation
message by @tiyash-basu-frequenz in AddMetricAggregation
message #22New Contributors
EVChargerType
enum variants #21Full Changelog: v0.2.0...v0.3.0
This discussion was created from the release v0.3.0.
Beta Was this translation helpful? Give feedback.
All reactions