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

Service Client example missing files #273

Closed
Hackerman342 opened this issue Jan 29, 2024 · 2 comments · Fixed by farm-ng/farm-ng-amiga#176
Closed

Service Client example missing files #273

Hackerman342 opened this issue Jan 29, 2024 · 2 comments · Fixed by farm-ng/farm-ng-amiga#176
Assignees

Comments

@Hackerman342
Copy link
Collaborator

📚 Documentation Issue

Reported by user https://discourse.farm-ng.com/u/gsainsbury86 in discourse thread: https://discourse.farm-ng.com/t/getting-started-with-brain-adk-and-additional-sensors/290/5

I got a bit stuck today. When trying to run the example Service Client demo, I got an error about not being able to find the two_ints_pb2 module. Reading the documentation, it says that these were the included protocolbuf files, but we could regenerate them with genprotos.py. I couldn’t find the original files or the genprotos script. After some reading, I found that I could generate the file using protoc --python_out=. two_ints.proto . Just a note in case anyone else gets stuck.

After that, I was able to define a new protobuf file for image_data passing and get a minimal example working where I pass an existing image between a client and service.

Service Client Example is missing two_ints_pb2.py, two_ints_pb2.pyi, & genprotos.py

@Hackerman342
Copy link
Collaborator Author

@edgarriba maybe these were omitted by the .gitignore?I looked around in farm-ng-amiga & farm-ng-core for genprotos.py and did not find it

@edgarriba
Copy link
Member

fixed farm-ng/farm-ng-amiga#176

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

Successfully merging a pull request may close this issue.

2 participants