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.

Sign up for GitHub

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

Jump to bottom

ghz-web convert timeunit error #181

Closed

forestbupt opened this issue Apr 21, 2020 · 1 comment

Assignees

Labels

WEB

forestbupt commented Apr 21, 2020

I believe there is a minor mistake in the function createChartData, it caused the time below 1ms always be displayed as 0, instead of x ns.

if (testValue > 1000000) {
unit = 'ms'
divr = 1000000
testValue = testValue / divr
}

if (testValue > 1000) { ######### it should be 1000000000 #########
unit = 's'
divr = 1000000000
}

bojand self-assigned this

bojand added the WEB label

Owner

bojand commented May 13, 2020

Hello, thanks for reporting this. It should be addressed in 0.54.0. Feel free to reopen if there are issues.

bojand closed this as completed

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