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

Getting available signals from broker #568

Closed
eriksven opened this issue May 24, 2023 · 5 comments
Closed

Getting available signals from broker #568

eriksven opened this issue May 24, 2023 · 5 comments
Labels
API-migration Databroker Issues related to databroker (core)

Comments

@eriksven
Copy link
Member

Hi, I would like to write a program which first requests all available signals in the data broker and then reacts based on that. The general idea is then to subscribe to all available signals and write any changes to a CSV-file.

I manually managed to achieve this goal with the 'sdv.databroker.v1' interface using the databroker-cli.
But for my solution, I am using the Python SDK from the kuksa-client through the kuksa.val.v1 interface which seems to have a different behavior here.

kuksa.val.v1:

Using something like

client.get(entries=[EntryRequest('', View.ALL, (Field.UNSPECIFIED,))])

results in ' ({'code': 404, 'reason': 'not_found', 'message': 'Path not found'}, [{'error': {'code': 404, 'reason': 'not_found', 'message': 'Path not found'}}])'. Using '' or something like 'Vehicle.' does not work either.

sdv.databroker.v1:

With the databroker-cli which seems to use the sdv.databroker.v1 interface, running

metadata *

returns a list of all available signals. The proto definition for the GetMetadataRequest states: "If no names are provided, metadata for all known data points will be returned.". This is exactly what I am looking for in the other interface.

@lukasmittag lukasmittag added API-migration Databroker Issues related to databroker (core) labels May 25, 2023
@SebastianSchildt
Copy link
Contributor

Related #411

@doosuu
Copy link

doosuu commented Jun 30, 2023

The mock service currently uses both the old and the new IF just to query all metadata. Would be greatly appreciated if the kuksa.val.v1 would also support this.

@SebastianSchildt
Copy link
Contributor

this is also related to #411 and @rafaeling started looking into this recently.

@rafaeling
Copy link
Contributor

Here is the draft PR for autocompletion: #605

@lukasmittag
Copy link
Contributor

Done since #605 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API-migration Databroker Issues related to databroker (core)
Projects
None yet
Development

No branches or pull requests

5 participants