Skip to content

Proposed method for interpolation of HRTFs in the time domain #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 8, 2019

Conversation

VeraE
Copy link
Member

@VeraE VeraE commented Mar 25, 2019

This is a proposal for an alternative HRTF interpolation method in the time domain:

The difference in TOAs is estimated by the maximum of the cross-correlation function. The HRIRs are then shifted to have the same TOA, interpolated and then the resulting HRIR is shifted to the interpolated TOA.
The method can handle an arbitrary number of HRIRs as input which is necessary depending on the used method for point selection.
I also added this additional method in test_interpolation_methods().

What do you think?

@VeraE VeraE requested a review from hagenw March 25, 2019 15:45
Copy link
Member

@hagenw hagenw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I have only a few comments.

In addition, I still have to test it under Octave, which I will do during the next days.

Copy link
Member

@hagenw hagenw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your changes look good and the test_interpolation_methods runs fine beside the two last plots, but those also fail under Octave for the master branch.

I had also a brief look at the new interpolation and could follow all the steps beside the last one, where I added a small question.

@VeraE
Copy link
Member Author

VeraE commented Apr 8, 2019

Do you know what the problem is with the last two plots in test_interpolation_methods() in Octave? I could not see anything special in the code that is different from the plots before.

@hagenw hagenw requested review from hagenw April 8, 2019 14:36
@hagenw
Copy link
Member

hagenw commented Apr 8, 2019

It was a strange openGL float data exception.
I will create an additional issue for that.

@hagenw hagenw merged commit 67d2034 into master Apr 8, 2019
@hagenw hagenw deleted the HRTF_timedomain_interpolation branch April 8, 2019 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants