Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@zhangzdd
Copy link
Contributor

@zhangzdd zhangzdd commented Jun 4, 2025

As part of the OSPO internship, I created the notebook for a LQR example based on the legacy website post (https://gtsam.org/2019/11/07/lqr-control.html).

@dellaert
Copy link
Member

I think we discussed some changes in our meeting?

…nk to colab runtime. Added visualization of the factor graph and Bayes net (obtained from different elimination ordering).
@zhangzdd
Copy link
Contributor Author

I have made several refinements to the previous version, including:
Added much richer mathematical explanation text to the code, providing more details on LQR problem itself and how GTSAM views the problem. Added link to colab runtime. Added visualization of the factor graph and Bayes net (obtained from different elimination ordering). Added a title.

@dellaert
Copy link
Member

dellaert commented Jul 5, 2025

I still think the ordering should be reversed and match the blog post, to indicate the DAG encodes policies P(u|x)

…n ordering in visualization and markdown text
@dellaert
Copy link
Member

dellaert commented Jul 9, 2025

The ordering in cell 20 is still wrong :-/

Copy link
Member

@dellaert dellaert left a comment

Choose a reason for hiding this comment

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

I know you fixed the ordering in our meeting but maybe you did not commit/push yet?

@dellaert dellaert merged commit 0794afb into borglab:develop Jul 30, 2025
9 checks passed
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