search

jensrossbach / node-red-contrib-sony-audio

Node-RED nodes for accessing Sony Audio Control API
MIT License
2 stars 0 forks source link

A little help needed! #1

Closed skavan closed 4 years ago

skavan commented 5 years ago

Hi, Cool node. Just a bit stuck on what to input to the controller to cause it to send one of the Filter commands.

i.e. I want to set up some manual triggers that do a:

What would help is a test flow to import...I should be able to deduce how it works from there.

Thanks.

skavan commented 5 years ago

OK..figured it out. A change node, with msg.command set to getSource or getSoundSettings etc...

Would be nice to not have to use a change node and use an Inject Node directly, but because "command" is a child of message, that doesn't work. Would it be possible to also allow an overridden command as a child of payload? i.e. payload.command. Shouldn't break anything!

What I have really wanted to get reported back from a STR-1080dn, is the input format of the video and audio signals (i.e. 4K 60 4:4:4 or 5.1 etc...) -- but it seems this isn't supported by the api - or if it is, it's not documented anywhere I can find!).

s.

jensrossbach commented 5 years ago

Hi,

yes if you want to do this programmatically, you need a change node. I attached an example with your use cases, one time with a dedicated controller node for each and another time with only one controller node and change nodes.

Putting the command directly inside msg made most sense for me to clearly separate the command from its content/arguments (= payload). The command is more like metadata while the arguments are the actual payload.

issue_1.zip

jensrossbach commented 5 years ago

If you want, you can create a feature request issue demanding to also accept a command property within msg.payload.

jensrossbach commented 5 years ago

@skavan I don't have a STR-1080dn, just a HT-XT3, so I can't test this. From what I can see in the API documentation, getPlayingContentInfo might give you at least some information.

jensrossbach commented 4 years ago

I decided to implement the feature requested by @skavan above. I will reuse this issue to track the progress.

jensrossbach commented 4 years ago

With the idea of supporting msg.topic and the ability to provide the command as topic (see issue #3), the implementation as described above will not be necessary anymore. Inject nodes provide the possibility to set the topic of the output message, so no change node would be needed anymore. Therefore I will close this issue.