-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use mypy #252
Conversation
Coverage reportThe coverage rate went from
Diff Coverage details (click to unfold)curvesim/logging.py
curvesim/pool/base.py
curvesim/pool/cryptoswap/pool.py
curvesim/pool/stableswap/metapool.py
curvesim/pipelines/simple/strategy.py
curvesim/pool/snapshot.py
curvesim/iterators/price_samplers/price_volume.py
curvesim/pool/stableswap/pool.py
curvesim/pool/cryptoswap/calcs/init.py
curvesim/templates/init.py
curvesim/templates/log.py
curvesim/pipelines/vol_limited_arb/strategy.py
curvesim/pool/cryptoswap/calcs/factory_2_coin.py
curvesim/pool/cryptoswap/calcs/tricrypto_ng.py
curvesim/templates/strategy.py
curvesim/metrics/state_log/log.py
|
299e415
to
f90e06d
Compare
Description
Closes #253. Makes progress toward #162.
As part of fixing
mypy
errors, ended up sneaking a new interface in (see changelog entry).TODO:
Hygiene checklist
(modules, public functions, classes, and public methods)
and descriptive commit messages following Tim Pope's style
Cute Animal Picture