[ossia-max] ossia.router bug with numbers as addresses

[navid]

Funny behaviour happens when trying to route the following message: servos/y2/3 11.5

If i route servo/y2
then ossia.router returns:
"3" 11.5
whereas it should return:
3 11.5

however I can still route servos/y2/3
in which case the router behaves correctly and returns:
11.5

[petervanhaaften]

i have tested and confirmed this bug on the Ossia "2022-1-25-38" build.
The issue appears to be, when a list is sent, the first value is routed as a symbol. With lists with more values than two, the first value still returns as a symbol, the following values still return as a list.
An examples patch based on Navid's screenshot is attached.

router-bug.maxpat.zip

[jcelerier]

is that the expected behaviour ?

[jcelerier]

hm there's still something that I don't understand: surely in servos/y2/3, "3" is supposed to be a symbol and not converted to a number, right? (the only way tomake it show up without the quotes in max).
I checked and j.oscrout definitely does the same:

(and I don't see any other way to do it sanely: for instance imagine that you have an OSC address such as:

/audio/5.1/volume

that you want to route: if we convert it from floating point to string (or conversely) for instance it could end up as "5.100000000000001" or something like this due to the way FP works, so we really have to stay with symbols all the time

[jcelerier]

(and even if Max displays "5" in the message box, the actual data we receive in the object is the string 5 so everything is fine)

[jcelerier]

ossia-max-2023-01-10-15-55.zip

here's a test version @navid @petervanhaaften

[jcelerier]

router.maxpat.zip

[jcelerier]

@navid @petervanhaaften tell me when you have tested that I close the issue :)

[navid]

thinking about it still. The issue arises out of the fact that ossia and jamoma routers have removed the slash from the bending of the osc addresses, thus rendering and osc outputs such as /3 x into instead “3” x within max when parsed as such. This behaviour is inconsistent with all the other osc routers out there (cnmat, ircam, etc) yet i guess I see why ossia and jamoma at some point decided to remove the leading slash from their addresses. Let’s sleep on this for a another week: since an optimal solution doesn’t seem to exist and any solution would be a compromise of some sort

On Thu, Jan 12, 2023 at 3:10 PM Jean-Michaël Celerier < ***@***.***> wrote: @navid <https://github.com/navid> @petervanhaaften <https://github.com/petervanhaaften> tell me when you have tested that I close the issue :) — Reply to this email directly, view it on GitHub <#802 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABDTZMWMCFDE4PNL3WO4GDWSBQL3ANCNFSM6AAAAAARQUKGYE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

--

…

___________________________________________ Multidisciplinary Composer | Media Alchemist | Artist-Researcher Associate Director, Topological Media Lab Research Associate: Matralab, Milieux, HEXAGRAM, LASG navidnavab.com <http:https://navidnavab.com> | 514.432.6633 ----------------------------------------------------------------------------

[jcelerier]

what about ossia.router staying like this to not break compat with existing patch, and introducing an ossia.oscrouter which would add the /

[navid]

that is a good solution. another alternative solution could be an attribute like @leading-slash 1 (default = 0)

On Thu, Jan 12, 2023 at 4:30 PM Jean-Michaël Celerier < ***@***.***> wrote: what about ossia.router staying like this to not break compat with existing patch, and introducing an ossia.oscrouter which would add the / — Reply to this email directly, view it on GitHub <#802 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABDTZNBFKDN2YZT4REOVIDWSBZYNANCNFSM6AAAAAARQUKGYE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

--

…

___________________________________________ Multidisciplinary Composer | Media Alchemist | Artist-Researcher Associate Director, Topological Media Lab Research Associate: Matralab, Milieux, HEXAGRAM, LASG navidnavab.com <http:https://navidnavab.com> | 514.432.6633 ----------------------------------------------------------------------------

[jcelerier]

ossia-max-2023-01-20-17-39.zip

latest version with the test patch:

----------begin_max5_patcher----------
1700.3oc0a0sTaqCD95vSgGeMMHsRxxtuJG5zwD7o0cH1Yhc5gNc569wVqMn
.DGUsRLjafYcLnuuc09qT98UqRuq8wptzjOm7OIqV86qVsx7nwGrZRdU51xG
27PYm40R2V00U9spzqwOq4v15lGp5MeH77CaOzO+T9zSqu27On8te7IY17eO
9d8+ZWEBhzzjuL8Q6J62785lu808Ua5wOMSkslccBmKF+khM9S.VyR9x3exe
t5pwebseLou5QypjdyufaDCqwZ07G8PcS0l1CM8ufj9vbleLWIJdEyEp.y7t
p8+rsaj+pDNHnYiyD9wzB.IWtd7WEhnXieIECnENC7j2Yl81JddDsv16p8jd
b+nWdAxKdV3cc8jId5JlyYgiIMU+2.Vdkc5Gqa61re.WUIiQi9KHJ7ZhpJNM
Qu9rdil3NxbCMQexf6Mdap51T5QbTdlUQqf2WCJQyoveyoNWXYN0wwb9bZji
Si5YFSomVUjpy4QxhrUEIMMKqL.VVI2P2hnXYe7FpFSOSNxYbXMnFnj.cOKV
qh.+T27HQ9w8M4O5JFD1chMq8sc+Z6csO7lLj6JCEE9VVmYiInwfrwIxCwcm
hbZbKp6MI66Ij9V4FVKJmGwltrZ83lGu4eaaSJnwVOMkBU9n0iqTl1sXEwls
IRfFQ4JZLkqiXkcOSTYBjSinflDQyydW3oHQnIxSZackEuK7b1MEnVeG3YU6
.1F1rmJONdpgvA0yLlpLSGyRlLlC6J.7yS+RPphO+rizBLp6U4BZ6UwXsEww
2jZ7UtmyCgmqLwchpc71T43HB3rnLUVvyx103HC3X6VeXmYG3YWWJswjx0p3
MXj1tt5x0ldn2GfYcADZgVUfoRjx.3k5FcoQVNmv7BjZjrp2ox+Nx20yA74o
WJCSz.XI8vEvPu.O6USgy4BoZbl40Q6fCwTu.EEWVrp9oiEjcIrKFXjJhXhq
QZaL4hH7r4kwIRG8hHn1YF2yVUD3oX9AmbdFxgCYgibtDy41TwPoflBBeSFK
dSFKdCFCKG34bAejfgxBMlXoPFmNznOQAOa.UpvSPgoe2J+iVlDs+IRDXQef
JlSOgpczyvObMNaj772I6XHpJfPosycaWTDhpBbgvhDIsPQRZQh334oLGIRj
eAzolfrAFvysu3R3TrAR07gyzkX87k886OTOiiQIyienp79gE8ScCu32oc1f
vSIZ1Utuba0vFkuV0Td2CFtx7TCXHMWiGKC6CtJPvhkJ.mc3EfFfGdMf.CuA
rKDU.DdUfRwQcP1EhNPDdcPtzJZ3GeUfTGAOgo5TgE0Al+ClAj+h61s4+33y
OVwz0dX+l40e5zySXOgi6q55qaJ6qaardG9Quy15620V2zOsfB7vq33HUAt4
NQXjNgwxUjoc.YCogR3mDYCc0Ld+Lv10jEOK8zeQ696qL6FXQGpxEUhHTEX0
c7rrmjdMTgnC0w1kcEqRgXArxIg0wKV940qhk.aAdGcjXvTcA7jDwcmpi258
1Xa7Fhe58mbFyXmU3.UNRLDn6bZtwK38oQWgH2Bb1RTwlvEMGaIrkqXVXyVJ
DX6b5swae7ow1zdrIrYKQEaN.MPsjyflKGAC90OHiolEHBLoKwTldGZKjjb7
f4H.3UbyVhJ1bZmSgKXSa6wYjBc9J2.qSJxLkshLSE9r.RvAvJVTyxYCcVoG
y9qYnt0DwvHQ0tyIitBsItOf2uoYronWA0n6Bbtb85kBlA3M9PLMPBKo..sy
VFhbwrmSEIgPyVJ.P6bom3KldBvpMl0ZVRDglvkz5B4xo0MauDLaGAiDUrkS
2QvZqePcDDh+hbmzVI3un9ZZqD28lNnsPL2aQj1B4RMqK52gMeN8MOvVhHx.
m76BfJ.BU6334CHvuKwGID9F0Acnlg.VU1LPOVJ7vNiJrmAC9cryVhJzTNjC
RN4KPekNaTjiy1En55bZogEKBcpGlocJYL4SRgthYPRt6KqFtBY2WfS41VbP
DyJNzKLi+rDUrwBkKFZgmBB7BovGYHPnd95hndNHbzvLWEpfvxiB6JiaPXtH
T6PjGsmPF2cHlJ3NaqUEK6zka0+hsTHv1YafIeIrovChbBa1Rg.am0buXQdy
Cbd59uZIQEatTPMr3LS0pLq.o1RTwl6SXa4RimNe0oSFQnCelRt6kJuT8D3A
aUta2Oq12M8WaVyzsk+n0754WaDqaPQyksIce0Oqmee7Ik627859pM8G1Wge
+ZyvKgS511gEt4P8zZOv1gkzbtgMkaq51UhDyb7hW8mq9ebjkA9O
-----------end_max5_patcher-----------

[jcelerier]

ossia-max-24-04-2023-ossia-router-optim.zip

with some optimizations