frequenz-floss / frequenz-sdk-python

Frequenz Python Software Development Kit (SDK)
https://frequenz-floss.github.io/frequenz-sdk-python/
MIT License
13 stars 17 forks source link

Restructure timeseries module #704

Closed matthias-wende-frequenz closed 2 months ago

matthias-wende-frequenz commented 1 year ago

What's needed?

Some of the parts should move out of the module because they are not solely related to timeseries.

Proposed solution

Quantities

Data pipeline

pools

Alternatives and workarounds

Leave it as is.

Additional context

https://github.com/frequenz-floss/frequenz-sdk-python/issues/84 https://github.com/frequenz-floss/frequenz-sdk-python/issues/575 https://github.com/frequenz-floss/frequenz-sdk-python/issues/222

llucax commented 1 year ago

Yeah, imho everything that is working with the micrograms should be in the microgrid namespace.

Also I would move actors to their domain namespace, and leave actor for the actor framework stuff. This is also a source of a lot of cyclic imports.

llucax commented 2 months ago

Closing in favor of #575.