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

fixed bulletrave compilation as it used obsolete API #706

Open
wants to merge 5 commits into
base: production
Choose a base branch
from

Conversation

cielavenir
Copy link
Collaborator

by the way compilation on libbullet 2.83 worked, so you could update CMakeList as well

@rdiankov
Copy link
Owner

@cielavenir doesn't look like dofbranches is used anywhere, so why not just:

pinfo->pbody->GetLinkTransformations(vtrans);

?

@cielavenir
Copy link
Collaborator Author

@rdiankov pushed

@cielavenir
Copy link
Collaborator Author

#795 appeared...

@cielavenir
Copy link
Collaborator Author

-Wshadow broke compilation, fixed quickly

@cielavenir cielavenir changed the base branch from master to production September 15, 2022 02:08
@cielavenir cielavenir requested a review from ziyan September 15, 2022 02:09
@cielavenir
Copy link
Collaborator Author

may I ask how is this MR going?

@cielavenir
Copy link
Collaborator Author

@rdiankov thank you for checking #1336 . How about this - openrave build fails with libbullet installed.

@cielavenir
Copy link
Collaborator Author

cielavenir commented Jul 10, 2024

@felixvd @hemangandhi @ntohge bulletrave failed to follow CollisionReport. Now should I give up this long-lasting pull request?

like

285.2 /openrave/plugins/bulletrave/bulletcollision.h: In member function 'virtual bool BulletCollisionChecker::CheckCollision(const OpenRAVE::RAY&, OpenRAVE::KinBodyConstPtr, OpenRAVE::CollisionReportPtr)':
285.2 /openrave/plugins/bulletrave/bulletcollision.h:685:25: error: 'class OpenRAVE::CollisionReport' has no member named 'plink1'
285.2   685 |                 report->plink1 = GetLinkFromCollision(const_cast<btCollisionObject*>(rayCallback.m_collisionObject));
285.2       |                         ^~~~~~
285.2 /openrave/plugins/bulletrave/bulletcollision.h:689:25: error: 'class OpenRAVE::CollisionReport' has no member named 'contacts'
285.2   689 |                 report->contacts.push_back(CONTACT(p,n.normalize3(),report->minDistance));
285.2       |                         ^~~~~~~~
285.2 /openrave/plugins/bulletrave/bulletcollision.h: In member function 'virtual bool BulletCollisionChecker::CheckCollision(const OpenRAVE::RAY&, OpenRAVE::CollisionReportPtr)':
285.2 /openrave/plugins/bulletrave/bulletcollision.h:750:25: error: 'class OpenRAVE::CollisionReport' has no member named 'plink1'
285.2   750 |                 report->plink1 = GetLinkFromCollision(const_cast<btCollisionObject*>(rayCallback.m_collisionObject));
285.2       |                         ^~~~~~
285.2 /openrave/plugins/bulletrave/bulletcollision.h:754:25: error: 'class OpenRAVE::CollisionReport' has no member named 'contacts'
285.2   754 |                 report->contacts.push_back(CONTACT(p,n.normalize3(),report->minDistance));
285.2       |                         ^~~~~~~

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.

2 participants