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 ign-math6 as of Vector NaN constants to main #308

Merged
merged 10 commits into from
Dec 17, 2021

Conversation

scpeters
Copy link
Member

➡️ Forward port

Port ign-math6 as of 0d884db (#222) to main

Branch comparison: main...0d884db

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

WagnerMarcos and others added 10 commits August 30, 2021 12:02
…im#220)

* Adding %rename tag to interface files to match PEP8
* Applying alphasort to include and snake_case to variables

Signed-off-by: Marcos Wagner <[email protected]>
Signed-off-by: Marcos Wagner <[email protected]>
Signed-off-by: Franco Cipollone <[email protected]>

Co-authored-by: Marcos Wagner <[email protected]>
Co-authored-by: Franco Cipollone <[email protected]>
…m#224)

Signed-off-by: AmrElsersy <[email protected]>

Co-authored-by: Ashton Larkin <[email protected]>
Co-authored-by: Steve Peters <[email protected]>
Signed-off-by: Marcos Wagner <[email protected]>

* Kmeans_TEST.py: remove duplicate code

Signed-off-by: Steve Peters <[email protected]>

Co-authored-by: Franco Cipollone <[email protected]>
Co-authored-by: Steve Peters <[email protected]>
* Adds scripting interface to Color and a python test.
* Adds scripting interface to Spline and a python test.
* Adds python support for operator[] and operator<< to the Color class.
* Adds %rename tag to interface files in order to match pep-8 naiming style.

Signed-off-by: LolaSegura <[email protected]>

Co-authored-by: Steve Peters <[email protected]>
Co-authored-by: Franco Cipollone <[email protected]>
@codecov
Copy link

codecov bot commented Dec 17, 2021

Codecov Report

Merging #308 (9c39ee4) into main (e0cc08f) will increase coverage by 0.19%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
+ Coverage   99.08%   99.27%   +0.19%     
==========================================
  Files          66       67       +1     
  Lines        6004     6077      +73     
==========================================
+ Hits         5949     6033      +84     
+ Misses         55       44      -11     
Impacted Files Coverage Δ
include/ignition/math/Vector2.hh 100.00% <ø> (ø)
include/ignition/math/Vector3.hh 98.84% <ø> (+3.08%) ⬆️
include/ignition/math/Vector4.hh 95.38% <ø> (+1.15%) ⬆️
eigen3/include/ignition/math/eigen3/Util.hh 100.00% <100.00%> (ø)
include/ignition/math/Line2.hh 100.00% <100.00%> (ø)
include/ignition/math/OrientedBox.hh 100.00% <0.00%> (ø)

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 e0cc08f...9c39ee4. Read the comment docs.

@scpeters scpeters merged commit 3c96971 into gazebosim:main Dec 17, 2021
@scpeters scpeters deleted the merge_6_nan_constants_to_main branch December 17, 2021 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants