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

Remove deprecated code 2021-04-21 #15050

Merged

Conversation

rpoyner-tri
Copy link
Contributor

@rpoyner-tri rpoyner-tri commented May 15, 2021

Missed previously, perhaps owing to idiosyncratic date.

  • drake::multibody::MultiBodyPlant::CalcCenterOfMassPosition
    • all overloads

This change is Reviewable

Missed previously, perhaps owing to idiosyncratic date.

* drake::multibody::MultiBodyPlant::CalcCenterOfMassPosition
  * all overloads
@rpoyner-tri rpoyner-tri added the status: single reviewer ok https://drake.mit.edu/reviewable.html label May 15, 2021
Copy link
Contributor Author

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+(status: single reviewer ok) +@jwnimmer-tri for both reviews.
FYI @mitiguy

Reviewable status: LGTM missing from assignee jwnimmer-tri(platform) (waiting on @jwnimmer-tri)

Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: both.

Reviewed 4 of 4 files at r1.
Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignee jwnimmer-tri(platform) (waiting on @rpoyner-tri)

@jwnimmer-tri jwnimmer-tri merged commit 3e86d88 into RobotLocomotion:master May 16, 2021
rpoyner-tri added a commit to rpoyner-tri/drake that referenced this pull request May 16, 2021
Missed previously, perhaps owing to idiosyncratic date.

* drake::multibody::MultiBodyPlant::CalcCenterOfMassPosition
  * all overloads
rpoyner-tri added a commit that referenced this pull request May 17, 2021
* DNM Disable CI

* notes: clean up the first few sections

* wip

* wip

* wip

* wip

* wip

* wip

* wip

* wip: first pass done

* Factor out to h2 and sort the Build Dependencies

* Remove non-notable changes (refactorings, docs)

* Move APT site to announcements

* Punt hydroelastic viz change into dynamics section

Probably all of the hydroelatic changes should be culled, but having
them all in once place is a start.

* Prioritize and reword the breaking changes section

* Note AVX breaking change

* Fix some pydrake typos

Remove non-notable (unit test) change

* quadrotor example: add getter for geometry frame (#15046)

* quadrotor example: add getter for geometry frame

* Enable numpy math functions to work with symbolic variables. (#15039)

* Enable numpy math functions to work with symbolic variables.

#15038

Co-authored-by: Soonho Kong <soonho.kong@tri.global>

* Remove deprecated code 2021-04-21 (#15050)

Missed previously, perhaps owing to idiosyncratic date.

* drake::multibody::MultiBodyPlant::CalcCenterOfMassPosition
  * all overloads

* wip

* wip

* integrate new commits from merge of master

* Merge remote-tracking branch 'upstream/master' into release_notes-v0.30.0

* Fix CaMel case typo

* wip

* set date

* Revert "DNM Disable CI"

This reverts commit 608e8ec.

* wip
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: single reviewer ok https://drake.mit.edu/reviewable.html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants