search

overlay-market / overlay-risk

Risk metrics for various data streams
MIT License
13 stars 4 forks source link

[WIP] Parallel metrics unittest #39

Closed atitawat-pol closed 2 years ago

atitawat-pol commented 2 years ago

Added a new test file name test_influx_metrics_univ3_parallel.py image

atitawat-pol commented 2 years ago

@deepsp94 Would you please review?

deepsp94 commented 2 years ago

Great job @atitawat-pol! 🙌🏼 I noticed in the diff that the contents of the following file have changed: tests/helpers/influx-metrics/uniswap_v3/get-price-cumulatives.csv I'd recommend saving the data you need for your tests in a new file. Just so that the tests we already have in the repo stay unaffected. Happy to hear if you have different thoughts :)