Skip to content

QPS Report Helper Functions - #90

Open
anothrNick wants to merge 6 commits into
dyninc:masterfrom
anothrNick:qpsreport_improvements
Open

QPS Report Helper Functions#90
anothrNick wants to merge 6 commits into
dyninc:masterfrom
anothrNick:qpsreport_improvements

Conversation

@anothrNick

Copy link
Copy Markdown
Contributor

Two new methods that format the CSV data from get_qps into a queries per day and queries per hour JSON format. These are not making use of any new APIs, just helper methods that format the data.

@derekargueta

Copy link
Copy Markdown

@anothrNick if there's interest in finishing this, the build is mostly failing on flake8 linter errors

dyn/tm/reports.py:132:80: E501 line too long (91 > 79 characters)
dyn/tm/reports.py:133:80: E501 line too long (107 > 79 characters)
dyn/tm/reports.py:165:80: E501 line too long (147 > 79 characters)
dyn/tm/reports.py:170:80: E501 line too long (99 > 79 characters)
dyn/tm/reports.py:171:80: E501 line too long (157 > 79 characters)
dyn/tm/reports.py:180:80: E501 line too long (153 > 79 characters)
dyn/tm/reports.py:187:20: E127 continuation line over-indented for visual indent
dyn/tm/reports.py:220:80: E501 line too long (106 > 79 characters)
dyn/tm/reports.py:264:80: E501 line too long (115 > 79 characters)
dyn/tm/reports.py:278:80: E501 line too long (98 > 79 characters)
dyn/tm/reports.py:287:80: E501 line too long (103 > 79 characters)
dyn/tm/reports.py:291:80: E501 line too long (103 > 79 characters)
dyn/tm/reports.py:297:80: E501 line too long (194 > 79 characters)
dyn/tm/reports.py:308:80: E501 line too long (95 > 79 characters)
dyn/tm/reports.py:315:80: E501 line too long (164 > 79 characters)
dyn/tm/reports.py:321:19: E127 continuation line over-indented for visual indent
dyn/tm/reports.py:349:80: E501 line too long (105 > 79 characters)
dyn/tm/reports.py:391:80: E501 line too long (115 > 79 characters)
dyn/tm/reports.py:405:80: E501 line too long (98 > 79 characters)
dyn/tm/reports.py:419:80: E501 line too long (102 > 79 characters)
dyn/tm/reports.py:425:80: E501 line too long (169 > 79 characters)
dyn/tm/reports.py:437:80: E501 line too long (94 > 79 characters)
dyn/tm/reports.py:442:80: E501 line too long (161 > 79 characters)

cc @Northover@Marchowes

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@anothrNick@derekargueta