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.

Sign up for GitHub

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

Jump to bottom

feat(Protobuf): creates protobuf packets, protobuf files generator and paths include in cmake, receiver code for tests, #27

Merged

ersaraujo merged 34 commits into main from add_protobuf

Feb 24, 2023

Member

ersaraujo commented Feb 20, 2023 •

edited

Loading

Description

Publish vision packages via protobuf using SSL-Vision messages

Adding SSL-vision proto files
Adding generator of protobuf files in CMake
Adding a library to send socket packets

Solves #12

TODO

Future modifications

Remove legacy files for change in cmakelist
Remove old clients
Change entities units from cm to mm (Issue change entities units from cm to mm #24) to remove temporary fix made in package protobuf

bmmuc and others added 13 commits

February 4, 2023 19:10


          added protobuf v0

4e76827


          fix: undefined reference

6a5ccf4

wip

d44985e


          fix: add Protobuf package on CMakeLists.txt

f38c2a5


          feat(protobuf): created protobuf packet

a5c9d79


          fix (protobuf): fix cm to mm

206bd48


          Merge branch 'change-origin' into add_protobuf

311b7d3


          fix field packet

494e167


          fix entities vector

5f8e3dc

wip

1ebbc67


          Fix protobuf generation

19232eb


          fix entities

f375a92


          removed unnecessary prints and comments

9cc053b

FelipeMartins96 reviewed

View reviewed changes

Dockerfile Show resolved Hide resolved

FelipeMartins96 reviewed

View reviewed changes

docker_run Outdated Show resolved Hide resolved

FelipeMartins96 reviewed

View reviewed changes

docker_run Show resolved Hide resolved

FelipeMartins96 reviewed

View reviewed changes

CMakeLists.txt Outdated Show resolved Hide resolved

FelipeMartins96 reviewed

View reviewed changes

src/Network/visionServer/server.h Outdated Show resolved Hide resolved

FelipeMartins96 reviewed

View reviewed changes

src/Network/visionServer/server.cpp Outdated Show resolved Hide resolved

Member

FelipeMartins96 commented Feb 21, 2023

sei que ainda ta em draft kkkkk, mas tava vendo a partir dos problemas que tu comentou hoje, consegui alguns avanços testando aqui e deixei comentado. Agora testando aqui eu recebo só o time amarelo, sabe sobre isso?

Member Author

ersaraujo commented Feb 22, 2023

sei que ainda ta em draft kkkkk, mas tava vendo a partir dos problemas que tu comentou hoje, consegui alguns avanços testando aqui e deixei comentado. Agora testando aqui eu recebo só o time amarelo, sabe sobre isso?

sei, tem um erro no no vision.cpp

ersaraujo and others added 9 commits

February 21, 2023 21:03


          Update docker_run

1f7c46c

Co-authored-by: Felipe Martins <35344539+FelipeMartins96@users.noreply.github.com>


          adjust server.h

3287eb4


          fix(server): fix team check

6109ba2


          added dependecies

9434c98


          added args for cam

502dd56


          removes changes

f6fc0cb

Co-authored-by: Felipe Martins <35344539+FelipeMartins96@users.noreply.github.com>


          Update src/Network/visionServer/server.h

72e61a4

Co-authored-by: Felipe Martins <35344539+FelipeMartins96@users.noreply.github.com>


          removes unnecessary files

7034fae


          include protobuf generation

0fb4a21

ersaraujo requested a review from bmmuc

February 22, 2023 00:59

ersaraujo assigned ersaraujo, FelipeMartins96 and bmmuc

FelipeMartins96 mentioned this pull request

Use glob to make cmake file cleaner #28

Open

FelipeMartins96 reviewed

View reviewed changes

.devcontainer/devcontainer.json Outdated Show resolved Hide resolved

FelipeMartins96 requested changes

View reviewed changes

Member

FelipeMartins96 left a comment

acho que só mudar o ip para um de multicast e podemos dar o merge

Member

FelipeMartins96 commented Feb 22, 2023

adiciona tambem o protobuf no installDepedencies e no readme pfv

ersaraujo added 3 commits

February 22, 2023 10:07


          fixed devconatiner

eb4f8d8


          adding .cache in gitignore

1f2a4cc


          Merge branch 'add_protobuf' of github.com:robocin/vss-vision into add…

ee8c9bc

…_protobuf

bmmuc previously approved these changes

View reviewed changes

ersaraujo added 2 commits

February 22, 2023 10:51


          Update multicast ip

f1e90b6


          update dependencies

49d5b86

ersaraujo dismissed bmmuc’s stale review via

49d5b86

February 22, 2023 14:00

ersaraujo requested review from FelipeMartins96 and bmmuc

February 22, 2023 14:12

FelipeMartins96 requested changes

View reviewed changes

client/python/receiver.py Outdated Show resolved Hide resolved

FelipeMartins96 and others added 6 commits

February 22, 2023 18:20


          add cmake extension to devcontainer

5b54a21


          remove useless and buggy position processing

2b941e7


          remove bug for findEnemies which was removed on previous commit

d2c7863


          update missed origin translation conversion values

38750f5

wip

4edfb9a


          Update client/python/receiver.py

41c76e8

Co-authored-by: Felipe Martins <35344539+FelipeMartins96@users.noreply.github.com>

ersaraujo requested a review from FelipeMartins96

February 23, 2023 00:49

Member

FelipeMartins96 commented Feb 23, 2023

tudo certo, mas esses commits da outra branch? não da problema no merge não?

FelipeMartins96 approved these changes

View reviewed changes

ersaraujo merged commit 53d4c75 into main

ersaraujo mentioned this pull request

Using same ports as SSL #18

Closed

FelipeMartins96 deleted the add_protobuf branch

March 15, 2023 16:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels