-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Hey @acphile , Thanks for submitting the PR
CI supported jobs: [windows-cpu, centos-cpu, windows-gpu, unix-cpu, centos-gpu, unix-gpu, clang, website, sanity, miscellaneous, edge] Note: |
I think we should integrate it in the performance benchmark. Currently there is not mechanism to monitor the results |
I'm moving it to benchmark scripts in #18252 |
@leezu the performance benchmark we are running is focusing on model level performance on end to end training/inference, this one is more related to opperf to test components |
Let's close this as it's moved to benchmark scripts in #18252 |
related to #18330
We think that "test_metric_perf" does not enforce anything and currently there are speed issues related to using mxnet.numpy. We should simply delete the test because it only prints the elapsed time.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments