Skip to content

Conversation

@zxybazh
Copy link
Member

@zxybazh zxybazh commented Jun 17, 2022

Pointed out by @sunggg that the description of number and repeat for evaluator configuration is not accurate, updated to a version more consistent with TimeEvaluator.

TimeEvaluator

@zxybazh zxybazh changed the title [MetaSchedule][Minor] Fix EvaluatorConfig Comments [MetaSchedule][Minor] Fix EvaluatorConfig Argument Description Jun 17, 2022
Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

Who did this amazing figure? We should really include that in our document!!!

@junrushao junrushao merged commit 2708b6c into apache:main Jun 17, 2022
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
…e#11766)

Pointed out by @sunggg that the description of `number` and `repeat` for evaluator configuration is not accurate, updated to a version more consistent with `TimeEvaluator`.

![TimeEvaluator](https://user-images.githubusercontent.com/3203174/174385966-74d3dbf6-dcca-43ea-9c0b-a91b4a281687.png)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants