-
Notifications
You must be signed in to change notification settings - Fork 977
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
Network Dashboard improvements and bug fixing #1289
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know what you're changing with the groupings, but I trust you :)
@simonebruzzechesse I'm trying to target Python 3.10 for everything in the repo. Would that make sense for you? |
Sure! Function code works with Python 3.10 (tested both locally and on CF), should I explicitly pin the required version in README or is it fine >= 3.10? |
>= 3.10 sounds good, but whatever you prefer. |
* update documentation and fix group by issue causing duplicated metric * fix format md * fix toc indentation * update required python version in README
The latter issue was causing duplicated metric to be written on Cloud Monitoring resulting in the following error:
One or more TimeSeries could not be written: One or more points were written more frequently than the maximum sampling period configured for the metric