search

microsoft / rats

Rats is a collection of tools to help researchers define and run experiments. It is designed to be a modular and extensible framework currently supporting building and running pipelines, integrating configs and services.
MIT License
13 stars 4 forks source link

rats.apps optimizations and dx #329

Closed ms-lolo closed 1 month ago

ms-lolo commented 1 month ago

trying to tackle some performance issues and improving the api for defining service groups by making group providers generators.

this api breaking change should only require return statements in service group providers to be switched to yield along with the necessary type annotation change to Iterator[T_ServiceType].

this pr also renames a few private details and makes rats.examples public. my goal for the examples package is to turn it into an app that allows others to register examples as they develop libraries.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 41.74757% with 60 lines in your changes missing coverage. Please review.

Project coverage is 61.83%. Comparing base (14a9dae) to head (88e58ce). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
rats-devtools/src/python/rats/ci/_plugin.py 0.00% 17 Missing :warning:
rats-devtools/src/python/rats/devtools/_plugin.py 0.00% 9 Missing :warning:
rats-devtools/src/python/rats/examples/_cli.py 0.00% 8 Missing :warning:
...ats-devtools/src/python/rats/amlruntime/_plugin.py 0.00% 5 Missing :warning:
rats-apps/src/python/rats/apps/_container.py 88.88% 4 Missing :warning:
rats-apps/src/python/rats/logs/_plugin.py 0.00% 3 Missing :warning:
...t/python/rats_test/apps/example/_example_groups.py 57.14% 3 Missing :warning:
rats-devtools/src/python/rats/docs/_plugin.py 0.00% 3 Missing :warning:
...ts-devtools/src/python/rats/kuberuntime/_plugin.py 0.00% 3 Missing :warning:
...ats-devtools/src/python/rats/stdruntime/_plugin.py 0.00% 3 Missing :warning:
... and 2 more
Additional details and impacted files | [Components](https://app.codecov.io/gh/microsoft/rats/pull/329/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | Coverage Δ | | |---|---|---| | [rats-apps](https://app.codecov.io/gh/microsoft/rats/pull/329/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `∅ <ø> (∅)` | | | [rats-devtools](https://app.codecov.io/gh/microsoft/rats/pull/329/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `∅ <ø> (∅)` | | | [rats-pipelines](https://app.codecov.io/gh/microsoft/rats/pull/329/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `∅ <ø> (∅)` | | | [rats-processors](https://app.codecov.io/gh/microsoft/rats/pull/329/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `∅ <ø> (∅)` | |

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