Comments on: • smbPitchShift Bumped Up to Version 1.2 http://www.dspdimension.com/admin/smbpitchshift-bumped-up-to-version-12/ Advanced Signal Processing Tutorials & Software Sat, 05 Jul 2008 16:33:06 +0000 http://wordpress.org/?v=2.3.3 By: Bernsee http://www.dspdimension.com/admin/smbpitchshift-bumped-up-to-version-12/#comment-236 Bernsee Sun, 16 Mar 2008 19:02:02 +0000 http://www.dspdimension.com/admin/smbpitchshift-bumped-up-to-version-12/#comment-236 Eventually, yes (after a very long time, however). I didn't want to complicate matters more than necessary by re-wrapping the values in gSumPhase, and it doesn't really have any consequence on the result because sin() and cos() are periodic. If you intend to use the code in an actual product you will want to re-wrap the phase to increase the precision of sin() and cos() which would otherwise deteriorate with time. --smb Eventually, yes (after a very long time, however). I didn’t want to complicate matters more than necessary by re-wrapping the values in gSumPhase, and it doesn’t really have any consequence on the result because sin() and cos() are periodic. If you intend to use the code in an actual product you will want to re-wrap the phase to increase the precision of sin() and cos() which would otherwise deteriorate with time.

–smb

]]> By: Quentin Lister http://www.dspdimension.com/admin/smbpitchshift-bumped-up-to-version-12/#comment-235 Quentin Lister Sun, 16 Mar 2008 16:49:56 +0000 http://www.dspdimension.com/admin/smbpitchshift-bumped-up-to-version-12/#comment-235 Doesn't the gSumPhase array increment out of control? Doesn’t the gSumPhase array increment out of control?

]]>