delvtech / agent0

Analysis & simulation repo for Delv
https://agent0.readthedocs.io/en/latest/
Apache License 2.0
49 stars 21 forks source link

add typing to BasePolicy.config #1439

Closed wakamex closed 4 months ago

wakamex commented 4 months ago

goal is to implement type narrowing all the way down to the lowest agent and policy level, through the use of generics

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.70%. Comparing base (416b9ce) to head (6419a98).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1439 +/- ## ========================================== - Coverage 80.78% 80.70% -0.08% ========================================== Files 104 104 Lines 6452 6452 ========================================== - Hits 5212 5207 -5 - Misses 1240 1245 +5 ``` | [Flag](https://app.codecov.io/gh/delvtech/agent0/pull/1439/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/delvtech/agent0/pull/1439/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech) | `80.70% <100.00%> (-0.08%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=delvtech#carryforward-flags-in-the-pull-request-comment) to find out more.

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

wakamex commented 4 months ago

closed because #1443 needs a lot more work than this