Add Distribution Report in a new tab #31

Merged
merged 1 commit into from Oct 13, 2015

Projects

None yet

2 participants

@abhishekshukla85
Contributor

distributionreport

@abhishekshukla85
Contributor

Thanks for the contribution @jmaxxz

@matteofigus matteofigus commented on an outdated diff Oct 11, 2015
templates/report.html
+ axesDefaults: {
+ labelRenderer: $.jqplot.CanvasAxisLabelRenderer
+ },
+ axes: {
+ xaxis: {
+ label: 'Response time (seconds)',
+ tickOptions: {
+ formatString:'%.3f'
+ }
+ },
+ yaxis: {
+ label: 'Percent of responses',
+ tickOptions: {
+ formatString: function(val){
+ console.log(val);
+ return '%.1f%';}()
@matteofigus
matteofigus Oct 11, 2015 Owner

Hi @abhishekshukla85, thanks for your PR. It's a good feature to have.
I tried to execute this with some benchmarks locally, and I can see some undefined in my console, probably due to this console.log. Can you investigate? This is the file:
https://gist.github.com/matteofigus/61cb3ac662467b9f6076

Thanks,
Matteo

Abhishek Shukla Add Distribution Report in a new tab
659fef8
@abhishekshukla85
Contributor

@matteofigus I missed it. Fixed now.

@matteofigus matteofigus merged commit 8a15888 into matteofigus:master Oct 13, 2015

2 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@matteofigus
Owner

Published in 0.2.4

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