search

usnistgov / SpectrumBrowser

ITL
12 stars 11 forks source link

Use volk in bin_statistics_ff for a 2x speed boost #156

Closed djanderson closed 9 years ago

djanderson commented 9 years ago

@msouryal,

I rewrote bin_statistics_ff to use volk after doing a bit of profiling on my flowgraph a couple of weeks ago.

Kind of hard to see in the screenshot, but I ran 10G samples through the original and the rewrite for a time of ~40s and ~21s respectively. It really improved the throughput.

bin_statistics_ff_volk_rewrite

msouryal commented 9 years ago

Great work, Doug! Excuse my ignorance on git, but what should I do, if anything, to incorporate the modified bin_statistics_ff in the master branch? Yesterday, I pushed updates to USRPSpectrumSensor including support for SSL socket connections to the server.

djanderson commented 9 years ago

@msouryal, all you have to do is click the "Merge pull request" button and then close the pull request... I'll go ahead and do that, I just wanted to give you a chance to review before I made any changes to your code.