Skip to content
This repository has been archived by the owner on Sep 21, 2021. It is now read-only.

Change @time to @btime in need for speed #315

Closed
jlperla opened this issue Nov 15, 2018 · 3 comments
Closed

Change @time to @btime in need for speed #315

jlperla opened this issue Nov 15, 2018 · 3 comments
Assignees
Milestone

Comments

@jlperla
Copy link
Member

jlperla commented Nov 15, 2018

And put in a tiny example paragraph at the top for BenchmarkTools.jl

Make sure that we tell people to always use the $ interpolation.

@jlperla jlperla added this to the v1.0 milestone Nov 15, 2018
@Nosferican
Copy link
Collaborator

For benchmarking @btime or @benchmark will do. I have been using the $ interpolation, but would be good to check the intro section has an example (with and without).

@jlperla
Copy link
Member Author

jlperla commented Nov 15, 2018

For sure. The issue with the interpolatio is that it is very subtle and easy to forget. Maybe we can get around to helping with JuliaCI/BenchmarkTools.jl#118 (comment) after the v1.0 launch of the lecture notes.

@arnavs
Copy link
Member

arnavs commented Nov 15, 2018

Done at bf9ab32.

@arnavs arnavs closed this as completed Nov 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants