Re: [Faudiostream-devel] [PATCH] implement meta.{serial,order}

2023-08-27 Thread Julius Smith
Ok great - sorry I didn't read the whole thread carefully - just a drive-by observation On Sun, Aug 27, 2023 at 11:08 AM Oleg Nesterov wrote: > On 08/27, Julius Smith wrote: > > > > > note that atan2(fTemp0, fTemp0) == %pi/4 > > > > That's only true if fTemp0 is positive. > > For negative fTemp0

Re: [Faudiostream-devel] [PATCH] implement meta.{serial,order}

2023-08-27 Thread Oleg Nesterov
On 08/27, Julius Smith wrote: > > > note that atan2(fTemp0, fTemp0) == %pi/4 > > That's only true if fTemp0 is positive. > For negative fTemp0, it's 5 * %pi / 4, or - 3 * %pi / 4, depending on your > choice of domain. Yes, and if fTemp0 == 0 then atan2() is undefined. And this means that the rece

Re: [Faudiostream-devel] [PATCH] implement meta.{serial,order}

2023-08-27 Thread Julius Smith
> note that atan2(fTemp0, fTemp0) == %pi/4 That's only true if fTemp0 is positive. For negative fTemp0, it's 5 * %pi / 4, or - 3 * %pi / 4, depending on your choice of domain. On Sat, Apr 29, 2023 at 9:42 AM Oleg Nesterov wrote: > Stephane, Yann, > > Can I suggest another feature? The patch is

[Faudiostream-devel] [PATCH 2/2] add internal.{lo,hi}

2023-08-27 Thread Oleg Nesterov
This comes in a separate patch to a) show that it is very easy to add the new primitives, and b) add some documentation. So the new lo/hi primitives act as undocumented lowest/highest but actually work. Say, both process = lowest; and process = internal.lo; output outpu

[Faudiostream-devel] [PATCH 1/2] implement internal.{id,rate}

2023-08-27 Thread Oleg Nesterov
NOTE: this patch touches compiler/parser/, so you obviously need "cd compiler/parser; make" before "make" to build the compiler. This patch adds the new keyword `internal` which returns the builtin environment with 2 primitives (so far): 1. id(sig) - sig->serial() 2. rate(sig

[Faudiostream-devel] [PATCH 0/2] implement the "internal" builtin environment

2023-08-27 Thread Oleg Nesterov
Another ping. And this time I am actually trying to push, sorry ;) See the previous (unfinished) discussion: https://sourceforge.net/p/faudiostream/mailman/faudiostream-devel/thread/20230429164124.GA26777%40redhat.com/#msg37837542 Let me resend both patches with some renames and cosmeti