Deprecate velocity, velocity_smothed, and last_movement_onset #623
Conversation
* Fix issue with trials and training table merge; clarify use of SessionLoader
…vement_onset function; update wheel documentation
|
Just curious - what is the reason for deprecating those functions? We ran them and included the one-off output in the NWB version of the initial BWM release - are you recommending we remove it from there as well? |
The velocity function was never really used by us (sorry for the miscommunication on that). It simply returns the diff of the wheel position. To reduce noise caused by imprecision in the recording, we filter the signal. We'd advise against using the unfiltered velocity for analysis.
If possible, it would be worth replacing the velocity with the filtered velocity version in the data. |
… reaction/response time
velocity,velocity_smothed, andlast_movement_onsetget_movement_onsetfunctionNB: Replacing velocity_smoothed with velocity_filtered in the wheel alignment code significantly changes the cross correlation output. We'll need to check if it's an improvement.