search

eunaeuna / GraphFrameworkOnGPUs

2 stars 0 forks source link

Where are Gunrock TC results in Figure 1? #6

Open jowens opened 5 years ago

ogreen commented 5 years ago

We were under the impression that there was a correctness problem with TC in Gunrock. As such we did not provide results.

jowens commented 5 years ago

@eunaeuna asked several weeks ago (maybe longer?) for how we run TC and @neoblizz told her at that time (he has the email where he told her). We are not aware of any correctness issues with our TC implementation, and have not been informed of any from @eunaeuna or anyone else on the paper.

From #15:

I understand @neoblizz sent an email several weeks ago that indicated how you could use Gunrock to get TC results. It was in dev branch, if I recall. (It is currently in the master branch, https://github.com/gunrock/gunrock/tree/master/gunrock/app/tc .) Where are the Gunrock TC results?

neoblizz commented 5 years ago

I was actually conversing with Zahra:

Muhammad Osama mosama@ucdavis.edu to Zahra Ronaghi zronaghi@nvidia.com

https://github.com/gunrock/gunrock/tree/master/dataset/test-scripts/grapl19 here you go. And if you'd like to use TC, just switch to dev or dev-refactor and use one of the scripts for cc or bc as an example. TC has no special parameters.

PS: The scripts directory is now changed to https://github.com/gunrock/gunrock/tree/master/scripts/performance

eunaeuna commented 5 years ago

Thanks @neoblizz! We can ask Zahra to execute TC in dev-refactor branch with script in the new directory in below?

https://github.com/gunrock/gunrock/tree/master/scripts/performance SETTING[0]=" --src=0 --iteration-num=10 --traversal-mode=LB_CULL --in-sizing=1.1 --queue-sizing=1.2 --quick" SETTING[1]=" --src=largestdegree --iteration-num=10 --traversal-mode=LB_CULL --in-sizing=1.1 --queue-sizing=1.2 --quick"

Let me talk to Zahra if she can do it and get back to you all.

neoblizz commented 5 years ago

@eunaeuna you can just do that in master now, dev-refactor will soon be deleted (everything was merged to master with the new release).

eunaeuna commented 5 years ago

That's great! Thanks!

jowens commented 5 years ago

@neoblizz we probably don't need all those SETTINGs, right?

neoblizz commented 5 years ago

No, just the binary and the market/rgg files:

/path/to/tc/binary market /path/to/dataset.mtx