Skip to content

Bumps in garden : ign-sensors7#216

Merged
chapulina merged 1 commit intomainfrom
ci_matching_branch/bump_garden_ign-sensors7
Apr 15, 2022
Merged

Bumps in garden : ign-sensors7#216
chapulina merged 1 commit intomainfrom
ci_matching_branch/bump_garden_ign-sensors7

Conversation

@methylDragon
Copy link
Contributor

Signed-off-by: methylDragon <methylDragon@gmail.com>
@methylDragon methylDragon requested a review from iche033 as a code owner April 14, 2022 21:37
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Apr 14, 2022
@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

1 similar comment
@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

@codecov
Copy link

codecov bot commented Apr 15, 2022

Codecov Report

Merging #216 (d50538d) into main (c71e15f) will increase coverage by 12.28%.
The diff coverage is n/a.

❗ Current head d50538d differs from pull request most recent head dfde409. Consider uploading reports for the commit dfde409 to get more accurate results

@@             Coverage Diff             @@
##             main     #216       +/-   ##
===========================================
+ Coverage   67.71%   80.00%   +12.28%     
===========================================
  Files          34        1       -33     
  Lines        3342       15     -3327     
===========================================
- Hits         2263       12     -2251     
+ Misses       1079        3     -1076     
Impacted Files Coverage Δ
src/Manager.cc
src/ImageNoise.cc
src/RgbdCameraSensor.cc
src/SegmentationCameraSensor.cc
src/GaussianNoiseModel.cc
include/ignition/sensors/ForceTorqueSensor.hh
src/CameraSensor.cc
src/RenderingEvents.cc
src/ImageDistortion.cc
src/Lidar.cc
... and 25 more

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 c71e15f...dfde409. Read the comment docs.

@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

2 similar comments
@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests and pass the flaky one please (:

@chapulina
Copy link
Contributor

The warning is coming from ign-rendering:

CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/ignition-rendering7/ignition-rendering7-config.cmake:94 (find_package):
  By not providing "Findignition-plugin2.cmake" in CMAKE_MODULE_PATH this
  project has asked CMake to find a package configuration file provided by
  "ignition-plugin2", but CMake did not find one.

  Could not find a package configuration file provided by "ignition-plugin2"
  with any of the following names:

    ignition-plugin2Config.cmake
    ignition-plugin2-config.cmake

  Add the installation prefix of "ignition-plugin2" to CMAKE_PREFIX_PATH or
  set "ignition-plugin2_DIR" to a directory containing one of the above
  files.  If "ignition-plugin2" provides a separate development package or
  SDK, be sure it has been installed.
Call Stack (most recent call first):
  /usr/share/cmake/ignition-cmake3/cmake3/IgnUtils.cmake:189 (find_package)
  CMakeLists.txt:69 (ign_find_package)

So merging!

@chapulina chapulina merged commit 29b8a57 into main Apr 15, 2022
@chapulina chapulina deleted the ci_matching_branch/bump_garden_ign-sensors7 branch April 15, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌱 garden Ignition Garden

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants