Skip to content

Improved thermal camera tutorial#270

Merged
ahcorde merged 4 commits intogz-sensors7from
ahcorde/improve/thermal_camera
Sep 20, 2022
Merged

Improved thermal camera tutorial#270
ahcorde merged 4 commits intogz-sensors7from
ahcorde/improve/thermal_camera

Conversation

@ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Sep 16, 2022

Signed-off-by: ahcorde ahcorde@gmail.com

🦟 Bug fix

Summary

Improved thermal camera tutorial. Related with gazebosim/garden-tutorial-party#573

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • 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 and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: ahcorde <ahcorde@gmail.com>
@codecov
Copy link

codecov bot commented Sep 16, 2022

Codecov Report

Merging #270 (cd12710) into gz-sensors7 (85f20cb) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           gz-sensors7     #270   +/-   ##
============================================
  Coverage        67.75%   67.75%           
============================================
  Files               35       35           
  Lines             3734     3734           
============================================
  Hits              2530     2530           
  Misses            1204     1204           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

#include <cstdint>

#include <gz/msgs.hh>
#include <gz/msgs/image.pb>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<gz/msgs/image.pb.h>?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of including all messages, we just include the one that is required fot this example

Copy link
Contributor

@iche033 iche033 Sep 19, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh I mean the .h file extension is missing

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ups ed35f82

}
```

Use this `CMakeLists.txt` to compile the code
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mention to save file as main.cpp? e.g.

Save the above code to a file named `main.cpp` and use this `CMakeLists.txt` to build it

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from iche033 September 19, 2022 18:39
@ahcorde ahcorde merged commit fefa24e into gz-sensors7 Sep 20, 2022
@ahcorde ahcorde deleted the ahcorde/improve/thermal_camera branch September 20, 2022 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌱 garden Ignition Garden

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants