Skip to content

Fix bug#59

Closed
kzm784 wants to merge 1 commit intorsasaki0109:mainfrom
kzm784:fix-bug
Closed

Fix bug#59
kzm784 wants to merge 1 commit intorsasaki0109:mainfrom
kzm784:fix-bug

Conversation

@kzm784
Copy link

@kzm784 kzm784 commented Jul 3, 2025

No description provided.

@rsasaki0109 rsasaki0109 self-requested a review July 3, 2025 09:35
@rsasaki0109
Copy link
Owner

Thanks for the PR. You mentioned it’s a bug fix—could you clarify specifically what the bug was and how this change addresses it?
Also, I noticed that odom_sub_ is commented out. Is this PR still a work in progress or was this perhaps submitted by mistake?

@kzm784
Copy link
Author

kzm784 commented Jul 3, 2025

I’ve reviewed several issues related to the bug and applied the patch that was proposed in one of them. Feel free to ignore this if it’s not relevant.

@rsasaki0109
Copy link
Owner

Thanks for the update. Could you please clarify specifically which issue or bug this fix addresses? It’s a bit hard to assess the impact without knowing the context, so sharing the rationale behind the changes would be really helpful.

@kzm784
Copy link
Author

kzm784 commented Jul 10, 2025

Since this pull request contains multiple bugs, I will split them into separate pull requests and submit them later. Could you please close this pull request?

@rsasaki0109
Copy link
Owner

Got it. I’ll close this PR now. Thanks for letting me know.

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