search

radical-cybertools / radical.utils

Utility classes and tools for various radical projects
Other
8 stars 6 forks source link

implement custom (de)serialization for json and msgpack #401

Closed andre-merzky closed 6 months ago

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 88.88889% with 17 lines in your changes missing coverage. Please review.

Project coverage is 62.09%. Comparing base (676792f) to head (38b82cd). Report is 86 commits behind head on devel.

Files with missing lines Patch % Lines
src/radical/utils/zmq/queue.py 82.60% 4 Missing :warning:
src/radical/utils/serialize.py 94.11% 3 Missing :warning:
src/radical/utils/typeddict.py 86.95% 3 Missing :warning:
src/radical/utils/config.py 50.00% 2 Missing :warning:
src/radical/utils/zmq/message.py 33.33% 2 Missing :warning:
src/radical/utils/zmq/pubsub.py 87.50% 2 Missing :warning:
src/radical/utils/json_io.py 87.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel #401 +/- ## ========================================== + Coverage 61.81% 62.09% +0.28% ========================================== Files 61 62 +1 Lines 6764 6830 +66 ========================================== + Hits 4181 4241 +60 - Misses 2583 2589 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

andre-merzky commented 7 months ago

I iterated more on this, the PR should be ready for another review. Thanks!

andre-merzky commented 6 months ago

Thanks @mtitov, applied as suggested.