-
Notifications
You must be signed in to change notification settings - Fork 230
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
Add number normalization test and address followup for getJsonObject #10800
Add number normalization test and address followup for getJsonObject #10800
Conversation
Signed-off-by: Haoyang Li <haoyangl@nvidia.com>
else: | ||
return float(x) | ||
for i in range(len(gpu_res)): | ||
assert math.isclose(json_string_to_float(gpu_res[i][0]), json_string_to_float(cpu_res[i][0])) |
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.
Nit: Add a comment like:
verify relatively diff < 1.0e-7
How long this test case will run? < one minute? |
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.
LGTM
Signed-off-by: Haoyang Li <haoyangl@nvidia.com>
Signed-off-by: Haoyang Li <haoyangl@nvidia.com>
Yes. We used to run a case locally with a very large length, so it was slow, now it run fast with default length. |
build |
closes #10688
closes #10646
A minor follow-up pr for getJsonObject
This pr: