search

CNMAT / CNMAT-Externs

CNMAT Max/MSP externals
Other
96 stars 11 forks source link

sdif-tuples: outputinterval is misleading #77

Open equilet opened 6 years ago

equilet commented 6 years ago

the outputinterval message requires a single atom (float), yet in the message list accessible from Max, it reads as variable.

ramagottfried commented 6 years ago

ah ok, interesting, looks like they changed it from "anything" to "variable" -- would you prefer to only allow floats?

if we did that then it would display (float) in the inlet's right-click contextual menu, but then it would complain if you sent an int in. Unfortunately there's no float/int option.

Alternatively we could probably make two separate functions one for int and one for float, but then it might list two separate messages in the contextual menu which doesn't seem like what we want.

equilet commented 6 years ago

If we're able to specify fractions of a millisecond, I think it's worth it to make it a float and call it a day. I'm happy leaving this for a future date as there are many more pressing issues to handle... Removing milestone

ramagottfried commented 6 years ago

also some legacy patches might send an int for this value so we shouldn't remove that functionality.