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

Transfer timings for CUDA Scan are wrong in results.csv database #42

Open
grahamlopez opened this issue Sep 12, 2014 · 4 comments
Open
Assignees
Labels

Comments

@grahamlopez
Copy link
Contributor

I fixed a bug today with these timings, which I believe makes those results in the database wrong. After my fix, the rates are now similar between cuda/opencl, whereas previously there was a 10^3 difference.
I'm throwing this out there for suggestions about what to do with the results database.

@jsmeredith
Copy link
Contributor

Seems like it should be easy enough to detect which ones are wrong, with
that size of magnitude error? If so, I'd be fine with just changing the
database.

@grahamlopez
Copy link
Contributor Author

I guess you're volunteering then to hunt down all of the devices that we
have run on and make updates? ;-)

On Fri, Sep 12, 2014 at 2:06 PM, Jeremy Meredith notifications@github.com
wrote:

Seems like it should be easy enough to detect which ones are wrong, with
that size of magnitude error? If so, I'd be fine with just changing the
database.


Reply to this email directly or view it on GitHub
#42 (comment).

@jsmeredith
Copy link
Contributor

Yeah, no. (Not possible anyway.) But I'll estimate the right values since we know the math behind the the error and update results.csv (marking them as estimates). Assigning to myself.

@jsmeredith jsmeredith self-assigned this Sep 12, 2014
@jsmeredith jsmeredith added the bug label Sep 12, 2014
@grahamlopez
Copy link
Contributor Author

You the man.
On Sep 12, 2014 4:41 PM, "Jeremy Meredith" notifications@github.com wrote:

Yeah, no. (Not possible anyway.) But I'll estimate the right values since
we know the math behind the the error and update results.csv (marking them
as estimates). Assigning to myself.


Reply to this email directly or view it on GitHub
#42 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants