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

Adding additional fields to the camera sensor message #201

Merged
merged 10 commits into from
Jul 29, 2022

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Nov 18, 2021

Signed-off-by: Nate Koenig nate@openrobotics.org

🎉 New feature

Summary

Adding fields to the camera sensor message so that it matches the fields in SDF.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Signed-off-by: Nate Koenig <nate@openrobotics.org>
@nkoenig nkoenig marked this pull request as draft November 18, 2021 00:51
@codecov
Copy link

codecov bot commented Nov 18, 2021

Codecov Report

Merging #201 (eed2a21) into main (24a7617) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #201   +/-   ##
=======================================
  Coverage   84.56%   84.56%           
=======================================
  Files           7        7           
  Lines         855      855           
=======================================
  Hits          723      723           
  Misses        132      132           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24a7617...eed2a21. Read the comment docs.

@chapulina chapulina added the 🌱 garden Ignition Garden label Nov 18, 2021
@chapulina chapulina changed the title Adding addition fields to the camera sensor message Adding additional fields to the camera sensor message Jul 25, 2022
@chapulina chapulina added the Breaking change Breaks API, ABI or behavior. Must target unstable version. label Jul 25, 2022
@chapulina chapulina added the enhancement New feature or request label Jul 25, 2022
Nate Koenig added 2 commits July 26, 2022 08:39
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
@codecov
Copy link

codecov bot commented Jul 26, 2022

Codecov Report

Merging #201 (58b9dd0) into main (39bbcdf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #201   +/-   ##
=======================================
  Coverage   84.96%   84.96%           
=======================================
  Files          10       10           
  Lines        1024     1024           
=======================================
  Hits          870      870           
  Misses        154      154           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39bbcdf...58b9dd0. Read the comment docs.

@nkoenig nkoenig marked this pull request as ready for review July 26, 2022 16:05
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
proto/gz/msgs/lens.proto Show resolved Hide resolved
proto/gz/msgs/lens.proto Show resolved Hide resolved
proto/gz/msgs/camerasensor.proto Show resolved Hide resolved
Nate Koenig and others added 5 commits July 29, 2022 14:24
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <nate@openrobotics.org>
@nkoenig nkoenig requested a review from chapulina July 29, 2022 23:08
proto/gz/msgs/camerasensor.proto Outdated Show resolved Hide resolved
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina enabled auto-merge (squash) July 29, 2022 23:25
@chapulina chapulina merged commit 22a600a into main Jul 29, 2022
@chapulina chapulina deleted the camera_sensor_update branch July 29, 2022 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking change Breaks API, ABI or behavior. Must target unstable version. enhancement New feature or request 🌱 garden Ignition Garden
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants