genexpr not working
I’m having trouble with a genexpr I’m writing. The genexpr is a simple zero cross detector. I want the gen~ to output a 1 until the input signal has a positive going zero cross at which point I want a single 0 sample output.
Here is the code:
// zero cross History n1(0); n = in1; crossup = ( n >= 0 && n1 < 0 ); // 0 on cross over, 1 otherwise inverse = 1-crossup; out1 = crossup; // this works as expected //out1 = inverse; // this doesn't work - it outputs a constant 1! n1 = in1;
Notice the subtraction used to calculate inverse. crossup outputs correctly (0 all the time with a single sample of 1 on zero cross), but inverse stays constantly 1.
Am I doing something wrong?
I'm guessing that because genexpr is 'typeless', the compiler is guessing at types behind the scenes; and crossup is probably evaluated to be a bool. When I try to subtract a 'bool' from 1 something goes wrong with internal type casting and it fails. This is a TOTAL GUESS of course. More likely, I'm just doing something silly.
In more testing, I’ve noted that this produces a negative of the signal
crossup = neg ( n >= 0 && n1 < 0 ) ;
But this ALSO fails to work
crossup = neg ( n >= 0 && n1 < 0 ) + 1.0;
There must be something happening when adding or subtracting numbers to bools..
There’s definitely something funny going on. You’re right that GenExpr is a typeless language and that types are inferred for code generation. In the case of gen~ though, everything is a double so there isn’t a bool. As far as I can tell, the fact that inverse is constant is a bug. We will have this resolved for the next release. Thanks for pointing it out.
On further investigation, the problem only happens when using history.
Actually, there is not bug. gen~ and GenExpr are working fine. I was fooled by scope~. If you hook a change~ to the inverse output, you’ll see that it’s working properly:
----------begin_max5_patcher---------- 1114.3ocyWssbaaCD8YpuBT7PZxLxJ.Th5RcSmou0G5GPmwtiGHPXIjPAvg. xVNYR916R.RJxHJJFGWY8htr6hEm8r2H+xf.7R8NgAi9MzMnffuLHHvIJWPP w+CvaX63ILiyLrgqSEeCOzqJkY4qkpU2kI3VuahlGNhLDENcV9Wiib+iNlLh f92hSIictRu7iWMqzSYL0Jg2CWQGEAGYTT0A3rDNWuU4thvBgpsajpDg07cB 0askRI4B+5fA4eLrmAnR7H.rRXYE6bWJluNGfUAt+RrOkVfYrQtRwRvUPtCl Ibd9WQN9I7H7xzxK5dsxpXabWC9OyjvczV7SqYtQ9Ym4z7qqMZg9LnkSl2mL YrKflQpR3cj2oQWbI9SFgTxbeJaQuhvP7qPPbehFbxQqRA0LKdHBuDX8NKVo yotvi5xpQjtJVozSWslxx.4VQ1cBEaYhyBxOckb3KXC9S7Dw2fHk7yziWRad Ba57tnsE+f83gu583gKH+.83yu3ZwEeNlw6L.ISbasV3FNOIp7y1BuH7qxhn UB0I2BMrW0pggtnjN2MOabm0pz+O2GEVGihrBNofT.eHSDOHxLRsplqCvrzz ZhCpcjbl7iZmilNrRjT4EQpDkIdPVd9PuPH+LnTYIQMarqpX1z7Om6G+GsX5 dlBRkq35Du6uAQFsXXyOpaXhl+IQbcffg1NkTklILBkkYKPTk5Xw8rsI16Zm Lap+dFWbzC2ZhK.uJSFqU4fnwIyEWdcvjsH+fs5QsyBEKskCaf3XqYIKKmgK l2GVpzp0IMUUkTgdBlRtgYEVoGrgjJmJ2jlI8SJpj4Wmr1vyzIIMbkWyCsnI Fx6bwixX6ZmupSkf4xzxT.thihkqDFaSYV1JSSIMdb55Ej0a1aHuql9lM9PC CJrtlVZomNy0EOg3RViO3QLO1LrtasOZ6cGs3GaHXP4fvhlsWddRpPz5JNXH ItAaz1Cp62q4WEPNEGRelbH44vgzWPNjqiE4V1IWMrG7E0OYjNwOYb7B+q4D cbNa74rtKrll7X1cO+kzX0YOgTz2Rd202ptUoPe.IUT2ugQFFy1TPxaQJze7 ADA8l2.1h9c3Wu6Zz6eO7sVgb1gzvZHH7QZKr85QoQbKLMOe0j.b.8pBm4bL .KJHrTTtirqkFzi5rOYPLCRrKE3UQryzPGjbd5ZTkswZgQ8qV2YPWgjVDXZ5 VKbdDWqfIuJKh9KtXhVDT3ySuW9LJJtWUKSny62LpIu1yn1uuKQpN13cWflq ucBzn2lwKapJ5.fJl5gLr5wJUUa+uoZBMpI0rVFG2bYqe+jIeaWb+Gz9bPLo GHdxECho8DwiegP79BkXA77PfG3OcGmAOSa60LUExEShGsYmde4LtbLLV92r +AW6VFr+dKdmB2M2r7z+JFeOkU7hFGRUM3pivSKZXyAbzg7yAuwSegyhdfF5 kEZBOavYZOfyryFZ5Sgy7yJZnm.MSunPCM5YBGXNvWG7eNV7N4C -----------end_max5_patcher-----------
Ahh! Thanks! I was fooled by
scope~ too. Sorry for the false-alarm.