Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • toolbox toolbox
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 12
    • Issues 12
    • List
    • Boards
    • Service Desk
    • Milestones
    • Requirements
  • Merge requests 4
    • Merge requests 4
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Institute of Technical Acoustics (ITA)
  • toolboxtoolbox
  • Merge requests
  • !2

Fix conversion SOFA to DAFF

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Vera Erbes requested to merge fix_SOFA_to_DAFF into master Apr 06, 2017
  • Overview 1
  • Commits 3
  • Changes 2

I suggest two changes to fix the conversion from SOFA HRIRs to DAFF files:

  1. The HRIRs are only zero-padded if their lengths are not divisible by 4.

  2. When the source positions of the SOFA file given in degrees are recalculated in radians, small numerical inaccuracies arise. This leads to counting too many angles as "unique" and therefore changing the grid of directions. The result is a dataset with a lot more HRIR directions than before (filled with nearest neighbour impulse responses). I suggest to use the default tolerance of uniquetol() instead of unique() to fix this.

What do you think?

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_SOFA_to_DAFF