tqchen opened a new pull request, #612:
URL: https://github.com/apache/tvm-ffi/pull/612
[TEST] Relax test_shared_dag_hash_scaling_not_exponential ratio to 4x
The timing-based assertion `t19 <= t18 * 2.0` was flaky at microsecond
scales. CI frequently measures t18~4μs and t19~8μs; at that resolution
any clock jitter can double the reading, pushing t19 over a 2x ceiling
without any algorithmic regression.
Raising the threshold to 4x removes the false-positive noise while still
catching true exponential blow-up: without memoization the ratio compounds
far past 4x within a few depth steps, so the test retains its intended
diagnostic value.
Failing CI example:
AssertionError: Unexpected super-linear scaling: d18=0.000004s
d19=0.000008s
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
