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

Merge gz-math7 ➡️ main #595

Merged
merged 23 commits into from
Jun 21, 2024
Merged

Merge gz-math7 ➡️ main #595

merged 23 commits into from
Jun 21, 2024

Conversation

scpeters
Copy link
Member

➡️ Forward port

Port gz-math7 ➡️ main

Branch comparison: main...gz-math7

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

azeey and others added 20 commits September 1, 2023 10:53
Signed-off-by: Addisu Z. Taddese <[email protected]>
…402)

Replaces the old variables with the new ones used by the FindPython3 module. Right now IgnPython cannot be used as it only searches for the interpreter but pybind11 also needs the development components, thus it provides a workaround. The Win32 PYTHON_LIBRARIES (PYTHON_DEBUG_LIBRARIES) variable was also removed as there is no equivalent in the new CMake module for getting the debug libraries.

---------

Signed-off-by: Bi0T1N <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Co-authored-by: Bi0T1N <[email protected]>
Co-authored-by: Jose Luis Rivero <[email protected]>
Co-authored-by: Steve Peters <[email protected]>
Co-authored-by: Addisu Z. Taddese <[email protected]>
Fixes #257

Signed-off-by: Michael Carroll <[email protected]>
* Use on `push` only on stable branches to avoid duplicate runs
* Update project automation

Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Merge ign-math6 ➡️  gz-math7
Signed-off-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Jagadeeshan S <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Merge ign-math6 ➡️  gz-math7
Signed-off-by: Addisu Z. Taddese <[email protected]>
Expose non-const reference to edges in Graph.hh
* Add workflow

Signed-off-by: Addisu Z. Taddese <[email protected]>
* Don't install python3-distutils on 22.04, 24.04
* Remove old ruby cmake code

Signed-off-by: Steve Peters <[email protected]>
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label May 22, 2024
Copy link

codecov bot commented May 22, 2024

Codecov Report

Attention: Patch coverage is 97.14286% with 1 line in your changes missing coverage. Please review.

Project coverage is 95.97%. Comparing base (e0193a5) to head (c32e96e).
Report is 3 commits behind head on main.

Current head c32e96e differs from pull request most recent head c6c775e

Please upload reports for the commit c6c775e to get more accurate results.

Files Patch % Lines
src/python_pybind11/src/MecanumDriveOdometry.cc 96.55% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #595      +/-   ##
==========================================
- Coverage   95.98%   95.97%   -0.01%     
==========================================
  Files         143      147       +4     
  Lines        9940    10122     +182     
==========================================
+ Hits         9541     9715     +174     
- Misses        399      407       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@scpeters scpeters force-pushed the scpeters/merge_7_8 branch from c32e96e to b79cd28 Compare May 24, 2024 20:04
Signed-off-by: Benjamin Perseghetti <[email protected]>
@scpeters scpeters force-pushed the scpeters/merge_7_8 branch from b79cd28 to a6bd3ae Compare June 19, 2024 06:13
@scpeters scpeters force-pushed the scpeters/merge_7_8 branch from a6bd3ae to c6c775e Compare June 19, 2024 06:18
@scpeters
Copy link
Member Author

I've updated and force-pushed

@scpeters scpeters merged commit ef2211c into main Jun 21, 2024
8 checks passed
@scpeters scpeters deleted the scpeters/merge_7_8 branch June 21, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏛️ ionic Gazebo Ionic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

10 participants