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.

Sign up for GitHub

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

Jump to bottom

Add timing to 'original' version #1

Closed
arporter opened this issue Feb 1, 2017 · 1 comment
Closed

Add timing to 'original' version #1

arporter opened this issue Feb 1, 2017 · 1 comment
Assignees
@arporter

Comments

@arporter
Copy link
Owner

arporter commented Feb 1, 2017

Silvia has committed the 'original' version of the tracer benchmark. In this issue we will add timing to this code using the dl_timer library.
@arporter arporter self-assigned this Feb 1, 2017
arporter added a commit that referenced this issue Feb 1, 2017
@arporter
#1 Add dl_timer as a git submodule
4c53ae1
arporter added a commit that referenced this issue Feb 1, 2017
@arporter
#1 Add Makefile and README
a69c884
arporter added a commit that referenced this issue Feb 1, 2017
@arporter
#1 Change constant args to SIGN to be double precision. Add use of dl…
c16a1e0 
…_timer.
arporter added a commit that referenced this issue Feb 3, 2017
@arporter
#1 Add details of git submodule to top-level README
5fd5a91
arporter added a commit that referenced this issue Feb 3, 2017
@arporter
#1 Mention need for dl_timer in local README
57dcf06
arporter added a commit that referenced this issue Feb 3, 2017
@arporter
#1 Correct relative link target in readme
a457b67
arporter added a commit that referenced this issue Feb 3, 2017
@arporter
#1 Make relative link a bit more precise
5d76520
@arporter arporter mentioned this issue Feb 3, 2017
Merged
arporter added a commit that referenced this issue Feb 3, 2017
@arporter
Merge pull request #2 from arporter/add_timing
8d2bc8f 
Fix issue #1 by adding timing code.
@arporter
Copy link
Owner Author

arporter commented Feb 3, 2017

Closing issue now pull request has been merged.

@arporter arporter closed this as completed Feb 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arporter arporter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@arporter