search

RussTedrake / underactuated

The course text for MIT 6.832 (and 6.832x on edX)
Other
701 stars 203 forks source link

Upgrade to Drake v1.28.0 #557

Closed jwnimmer-tri closed 3 months ago

jwnimmer-tri commented 3 months ago

This is my first time running these poetry-upgrade tools, so you might want to re-run them to make sure I got it right.

Besides the Drake pin bump and related poetry updates, this also contains two notebook fixes for the quadrotor:

FYI I noticed that hybrid.ipynb also uses the default robot builder timestep, so is no longer a continuous time plant. The test still passes, though.

FYI I only ran the test //..., I didn't manually / visually check for any other kinds of problems.

This change is Reviewable

RussTedrake commented 3 months ago

Thanks! My CI is a mess right now (multiple things broke this week and I'm planning to fix today).

We have pre-commit hooks on this repo (described in a shoddy way in Developers.md); when black gets updated in the requirements, we also want to update the version in .pre-commit.yaml (one of the pre-commit hooks does this automatically) and the linting CI in .github/workflows/main.yaml .

RussTedrake commented 3 months ago

I've pushed the fixes (master is now passing again), and rerun the failing tests.