search

TWOEARS / blackboard-system

Two!Ears Auditory Model - Blackboard system module
http://docs.twoears.eu/en/latest/blackboard/
GNU General Public License v2.0
3 stars 2 forks source link

newly created AuditoryFrontEndDepKS.getSignalBlock() produces an error #13

Closed ivo--t closed 8 years ago

ivo--t commented 8 years ago

follow-up on closed #3 . @ningma97 : "I just notice when I request multi-stream signals such as "time" signals, the newly created AuditoryFrontEndDepKS.getSignalBlock() produces an error. I added a not-so-elegant fix for now. Perhaps Ivo could have a look at this, please."

hagenw commented 8 years ago

@ningma97 could you please post a short code example in order to reproduce this bug and provide a link to the commit/changes you did in order to fix it?

ningma97 commented 8 years ago

if you request a time signal in DnnLocationKS()

requests{3}.name = 'time';
requests{3}.params = param;

and then in execute() do sig = obj.getNextSignalBlock( 3, obj.blockSize, obj.blockSize, false ); you will get the error, which is related to the fact that 'signalStream' in getSignalBlock() is a 2x1 cell array rather than an object in this case.

What I did is just to check if signalStream is a cell array and do things differently, which is not very elegant.

https://github.com/TWOEARS/blackboard-system/commit/69b06c0538afeabe56b8a3edd413770ab128198b#diff-95f6cc79ce0fa39d05181f9634dec42c

ivo--t commented 8 years ago

Cleaned solution.