Skip to content

Edited the slam_se2#3

Open
DavidCapek wants to merge 1 commit intoctu-vras:mainfrom
DavidCapek:main
Open

Edited the slam_se2#3
DavidCapek wants to merge 1 commit intoctu-vras:mainfrom
DavidCapek:main

Conversation

@DavidCapek
Copy link

Rewrote the third lecture notes

@peci1 peci1 self-assigned this Mar 17, 2025
Copy link
Member

@peci1 peci1 left a comment

Choose a reason for hiding this comment

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

Thanks a lot! I have two comments/questions in code.

Comment on lines +65 to +68
```
x* = arg max_{x_0...x_t} p(x_0, ..., x_t | z_1, ..., z_t, u_1, ..., u_t)
∝ arg max_{x_0...x_t} p(x_0, ..., x_t, z_1, ..., z_t, u_1, ..., u_t)
```
Copy link
Member

Choose a reason for hiding this comment

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

Would it be possible to use more embedded Latex instead of these code blocks? (applies to the whole document).

Copy link
Author

Choose a reason for hiding this comment

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

Will do

Comment on lines +103 to +109
```
||x_1 - x_0||^2
|
x_1
|
||x_1 - z_1^{GPS}||^2
```
Copy link
Member

Choose a reason for hiding this comment

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

Would you be interested in trying Mermaid for these graphs?

It should be possible to enable it in this repo following https://jackgruber.github.io/2021-05-09-Embed-Mermaid-in-Jekyll-without-plugin/ .

Copy link
Author

Choose a reason for hiding this comment

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

I didnt have the oportunity to get acquainted with mermaid yet but will try.

@peci1
Copy link
Member

peci1 commented Mar 28, 2025

@DavidCapek would you please prioritize finishing these lecture notes? The rules of the bonus task say the notes should be finished within 2 weeks after the lecture...

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