frequenz-floss / frequenz-core-python

Core utilities to complement Python's standard library
https://frequenz-floss.github.io/frequenz-core-python/
MIT License
0 stars 2 forks source link

Initial version of the code #10

Closed llucax closed 5 months ago

llucax commented 5 months ago
llucax commented 5 months ago

I derailed a bit while importing the non-default constructible part :grimacing:

There are still a couple more modules I want to include: math with is_close_to_zero() (but I want to write some basic tests) and collections with Bounds (but I want to get the PR finalized in the SDK first).

llucax commented 5 months ago

@daniel-zullo-frequenz thanks to your comment I found some issues when subclassing, the custom error wasn't used. I updated the PR to use a more consistent default exception (before we used 3 different ones depending on when it was raised) and to fix the issue with subclassing, but looking up for the exception to raise in the base classes if not found in the current class.

I will enable auto-merge.

llucax commented 5 months ago

Thanks, updated!