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

Fix use of compound mesh in FCL #1083

Conversation

Levi-Armstrong
Copy link
Contributor

Fix #1053

@rjoomen After looking through the code I went a slight different route to solve another issue with get the correct shape index. Do you mind testing this?

@Levi-Armstrong Levi-Armstrong force-pushed the bugfix/fcl-compound-mesh branch from ec9ef9d to 5cba2a9 Compare December 15, 2024 18:03
@rjoomen rjoomen force-pushed the bugfix/fcl-compound-mesh branch from 5cba2a9 to 211da43 Compare December 16, 2024 12:43
@Levi-Armstrong Levi-Armstrong force-pushed the bugfix/fcl-compound-mesh branch 2 times, most recently from 8c5cae1 to 79d5edb Compare December 16, 2024 13:28
@Levi-Armstrong
Copy link
Contributor Author

@rjoomen Hey, I did not realize you had pushed changes and I force pushed after I noticed my local clang tidy build failed. What changes did you make?

@rjoomen
Copy link
Contributor

rjoomen commented Dec 16, 2024

No changes, I just rebased on master using Github.

@rjoomen
Copy link
Contributor

rjoomen commented Dec 16, 2024

I'll look into this tomorrow.

@Levi-Armstrong Levi-Armstrong force-pushed the bugfix/fcl-compound-mesh branch from 79d5edb to 6d16ec5 Compare December 16, 2024 16:46
Copy link

codecov bot commented Dec 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.40%. Comparing base (6aabd9f) to head (73496d2).
Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1083      +/-   ##
==========================================
- Coverage   89.40%   89.40%   -0.01%     
==========================================
  Files         288      288              
  Lines       16304    16303       -1     
==========================================
- Hits        14577    14576       -1     
  Misses       1727     1727              

see 5 files with indirect coverage changes

@Levi-Armstrong Levi-Armstrong force-pushed the bugfix/fcl-compound-mesh branch from 6d16ec5 to 00a03ba Compare December 16, 2024 17:14
@Levi-Armstrong Levi-Armstrong force-pushed the bugfix/fcl-compound-mesh branch from 00a03ba to 1b035c7 Compare December 17, 2024 00:46
@rjoomen
Copy link
Contributor

rjoomen commented Dec 18, 2024

Seems like a better solution, and it works, thanks. I added a small simplification.

@Levi-Armstrong Levi-Armstrong merged commit 6757445 into tesseract-robotics:master Dec 18, 2024
13 of 15 checks passed
@Levi-Armstrong Levi-Armstrong deleted the bugfix/fcl-compound-mesh branch December 18, 2024 15:57
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.

CompoundMesh does not work with FCL
2 participants