-
Notifications
You must be signed in to change notification settings - Fork 16
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
Error consolidating UK data #41
Comments
Well caught, this was due to some change in England's historical data that was improved by @tomwhite and badly handled on my side. |
@boogheta I'm seeing the same error again.
But it seems it fails this time near the very end of processing the data, and doesn't affect any data that actually gets charted. |
Yes indeed, thanks again @aramse. |
@boogheta firstly want to thank you for maintaining this beautifully efficient graph for comparing COVID data across countries. It's been very helpful to me personally in keeping up more with just the numbers and less with the politics. Also wanted to mention that I'm actually running this on Google Cloud: see this link. This is equipped with automatic data refreshes, autoscaling, CI/CD, monitoring, and more. Let me know if you're interested in a detailed demo of how this is set up and/or you're interested in doing this for any other applications! (Also let me know if I should make any changes with the way I'm crediting yourself/your company in the banner of this page.) |
Hi @aramse,
Thanks in advance! |
Great will look to do make the needed modifications to comply here, thank you! |
Updated to better comply 🙂 |
Running the
consolidate_uk.py
file is returning the following error:The text was updated successfully, but these errors were encountered: