d1manson / waveform

browser-based GUI for working with extra-neuronal waveform data
MIT License
11 stars 5 forks source link

speed units bug #34

Closed d1manson closed 8 years ago

d1manson commented 8 years ago

speed doesn't take account of units_per_m (which is 10,000 rather than 100 for cm in a meter) and doesn't take account of pos sample frequency (which is normally 50hz). The two "bugs" cancel and give speed off by a factor of 2 (it comes out as being too large).

d1manson commented 8 years ago

resolved with https://github.com/d1manson/waveform/commit/f15c6f1e44cbd221b06cdd983090d0cc2e04539c