Skip to content
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

Move the benchmarks to Criterion #507

Merged
merged 2 commits into from
Aug 28, 2018
Merged

Move the benchmarks to Criterion #507

merged 2 commits into from
Aug 28, 2018

Conversation

lu-zero
Copy link
Collaborator

@lu-zero lu-zero commented Aug 26, 2018

It produces more information including pretty graphs and immediate speedup/slowdown evaluation compared to previous runs.

Solves #398 by other means.

@rzumer
Copy link
Collaborator

rzumer commented Aug 26, 2018

Seems fine to me as a dev dependency, but that means there is no chance of removing external dependencies for benchmarking, so I would like to have @tdaede's opinion.

Also, if you could produce a sample output that would be interesting to see.

@lu-zero
Copy link
Collaborator Author

lu-zero commented Aug 26, 2018

here the full output.

@tdaede
Copy link
Collaborator

tdaede commented Aug 27, 2018

That's actually really nice, ffmpeg and x264 internal benchmarks have similar functionality. I don't have any problem depending on criterion for this.

@lu-zero lu-zero merged commit ea40e92 into xiph:master Aug 28, 2018
@lu-zero lu-zero deleted the criterion branch August 28, 2018 03:56
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.

3 participants