-
Notifications
You must be signed in to change notification settings - Fork 20
Closed as not planned
Closed as not planned
Copy link
Labels
part:microgridAffects the interactions with the microgridAffects the interactions with the microgridresolution:invalidThis doesn't seem rightThis doesn't seem rightscope:breaking-changeBreaking change, users will need to update their codeBreaking change, users will need to update their codestatus:blockedOther issues must be resolved before this can be worked onOther issues must be resolved before this can be worked ontype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Milestone
Description
What's needed?
Now in frequenz.sdk.microgrid we have modules like: component_errors, component_states, component_type.
Instead of we could have module for each component category. The same as in frequenz-api
It would:
- Simplify findings features of the component. Now it is not clear what is where.
- Instead of implementing class BatteryErrorCode it would be possible to implement class ErrorCode in battery submodule and import it as battery.ErrorCode
- It would be more consistent with the api.
Proposed solution
No response
Use cases
Easier to implement attributes of the component message.
Easier for user to find correct type.
Easier to import code.
Alternatives and workarounds
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
part:microgridAffects the interactions with the microgridAffects the interactions with the microgridresolution:invalidThis doesn't seem rightThis doesn't seem rightscope:breaking-changeBreaking change, users will need to update their codeBreaking change, users will need to update their codestatus:blockedOther issues must be resolved before this can be worked onOther issues must be resolved before this can be worked ontype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Type
Projects
Status
Done