npadmana / DistributedFFT

6 stars 2 forks source link

Should we include the reference codes with the repository? #37

Open npadmana opened 4 years ago

npadmana commented 4 years ago

Currently, the repo does not have any of the reference benchmark codes, just the Chapel code. Should we include these with the repo?

@ronawho -- thoughts?

edit (after Elliot's comment) Probably a good idea -- that way we don't lose these codes and have a reasonable idea of how we compiled/ran them.

ronawho commented 4 years ago

Either links to the reference code or copies of the code would be fine with me.

ronawho commented 4 years ago

UPC version was added in https://github.com/npadmana/DistributedFFT/pull/55

I think we just need to add MPI reference, and maybe include a link to the UPC one somewhere (http://www.nersc.gov/assets/Trinity--NERSC-8-RFP/Benchmarks/Jan9/UPC-FT.tar)

And maybe apply the diff from https://github.com/npadmana/DistributedFFT/pull/55#issuecomment-554387127?