jzarnett / ece459

ECE 459: Programming for Performance
446 stars 137 forks source link

Lab 1 manual hyperfine command #56

Closed peter-dye closed 2 years ago

peter-dye commented 3 years ago

The hyperfine command in lab 1 manual seems to be missing single quotes around the command to be benchmarked. I believe it should be hyperfine -i 'target/release/lab1 verify 100.txt'. I would open a pull request but the manual is only provided as a PDF.

broehl commented 3 years ago

Thanks -- I'll update the manual for next term.

On Thu, Jan 21, 2021 at 5:10 PM Peter Dye notifications@github.com wrote:

The hyperfine command in lab 1 manual seems to be missing single quotes around the command to be benchmarked. I believe it should be hyperfine -i 'target/release/lab1 verify 100.txt'. I would open a pull request but the manual is only provided as a PDF.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jzarnett/ece459/issues/56, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABMJ55LT7Z4XUSVTZ4BHILS3CQ4DANCNFSM4WNSQSNA .

-- Bernie Roehl Mail: broehl@gmail.com

patricklam commented 3 years ago

Fixed for next year.