Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

set fixed seed for profiler #16155

Merged
merged 3 commits into from
Sep 24, 2019
Merged

set fixed seed for profiler #16155

merged 3 commits into from
Sep 24, 2019

Conversation

apeforest
Copy link
Contributor

Description

Use a fixed seed for profiler to reduce run-to-run variation.

@apeforest
Copy link
Contributor Author

@ChaiBapchya review please

Copy link
Contributor

@sandeep-krishnamurthy sandeep-krishnamurthy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ChaiBapchya ChaiBapchya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@apeforest apeforest force-pushed the perf/topk branch 2 times, most recently from 6c3764a to b2677c5 Compare September 16, 2019 20:40
@apeforest apeforest merged commit 1d4032d into apache:master Sep 24, 2019
@apeforest apeforest deleted the perf/topk branch September 24, 2019 17:50
drivanov pushed a commit to drivanov/incubator-mxnet that referenced this pull request Sep 26, 2019
* set fix seed for profiler

* update README with profiler option

* fix wording
larroy pushed a commit to larroy/mxnet that referenced this pull request Sep 28, 2019
* set fix seed for profiler

* update README with profiler option

* fix wording
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants