Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OSCQuery: Support for optional attributes? #197

Open
joreg opened this issue Oct 10, 2023 · 6 comments
Open

OSCQuery: Support for optional attributes? #197

joreg opened this issue Oct 10, 2023 · 6 comments

Comments

@joreg
Copy link

joreg commented Oct 10, 2023

i'm particularly interested in PATH_ADDED and PATH_REMOVED attributes for Server->Client communication as defined here: https://github.com/Vidvox/OSCQueryProposal#server---client-communication-attributes

i am sending those from my oscquery server implementation but they don't seem to have an effect. are those supposed to work?

here is what i send:

{"COMMAND":"PATH_ADDED","DATA":"/foo/bars"}
{"COMMAND":"PATH_REMOVED","DATA":"/foo/bars"}

apart from those, can you provide a list of which the optional stuff is supported in chataigne?

@joreg
Copy link
Author

joreg commented Oct 12, 2023

also can you explain what Sync does?

grafik

@benkuper
Copy link
Owner

Sync allows to not create the inner container data. When you are syncing a big structure without thousands of parameters, it's really nice to not have a giant structure to load and manage from the software.
So it's mainly for optimization and convenience.

As for the supported options, I implemented a basic implementation to work with most existing software, and most of them don't implement those functions.
But I actually recently implemented them as server in organic apps, so it would make sense to implement them in the OSCQuery module.

I'll add that to the roadmap. You can definitely keep it there and it should just work at some point with Chataigne :) Right now, you would just have to hit the "Sync Data" button.

@joreg
Copy link
Author

joreg commented Oct 13, 2023

one more: when i click the listen button (on above screenshot) i receive 2 LISTEN commands:

  • /foo/bars
  • /foo/sync

is the "/foo/sync" one on purpose? if so, what is it meant for?

@benkuper
Copy link
Owner

Yes, right now the LISTEN on Chataigne side is on the Container level, meaning that you can't choose to listen only one parameter in a container. It was mainly for ease of development and UI, and considering that most of the time it won't be a problem.
There is no actual limitation on the fact that it could be on a parameter level

@joreg
Copy link
Author

joreg commented Oct 13, 2023

but why send "/foo/sync"? since this is not a parameter on my server the only thing i can do with it is ignore it. but what would then be its usecase?

@benkuper
Copy link
Owner

aaah sorry.
yes it's a bug :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants