Open macanudo527 opened 5 days ago
Oh, wow, this looks like a breaking change probably in some of the dependencies. I am short on time in this period but I'll try to take a look when I get a chance. I think we may want to update some of the dependencies that have a fixed version in setup.cfg.
This seems to be the issue: https://github.com/David-Woroniuk/Historic_Crypto/issues/27 Pulling pricing data from Coinbase Pro or coinbase it seems, no longer works at least with historic_crypto.
Oh, interesting: thanks for digging. If Historic Crypto is indeed the source of the problem, we can deprecate it and just use the other price converters instead.
Yeah, I'm looking for a suitable replacment. It seems like all of them want to download the giant kraken file each time. I'm going to see if I can make it easier.
Unfortunately data loaders are inherently very brittle (more than I thought they would be when I started DaLI). There should be some industry standard to reduce these complexities and improve robustness of client software, but, alas...
It seems like a recent upgrade has caused the ODS tests to start failing. I'm having a hard time interpreting the error output.
There are a few more mypy and pylint errors that have also cropped up that I will try to address in a separate PR, but @eprbell can you look at this when you have free time? This is just one of the stack traces, but all the tests fail when running this on the main branch for me locally: