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

demo.launch does't work #6

Closed
k0suke-murakami opened this issue May 18, 2020 · 5 comments
Closed

demo.launch does't work #6

k0suke-murakami opened this issue May 18, 2020 · 5 comments

Comments

@k0suke-murakami
Copy link

k0suke-murakami commented May 18, 2020

Hi, thank you for sharing the amazing repo.
Here is what I found while trying to run your demo.launch

Description

A trajectory isn't generated after following the guidance in readme.md.

Code

commit: a95f0d0
Nothing has been modified including config file.

Steps to reproduce

  • Following instruction in readme.md

Console output

Screenshot from 2020-05-18 18-20-04

@k0suke-murakami
Copy link
Author

@LiJiangnanBit
Copy link
Owner

LiJiangnanBit commented May 18, 2020

Hi, I think the program failed because the start state was not within the range of the reference points (red).
This could fail:
2020-05-18 18-02-30屏幕截图.png
While this should work:
2020-05-18 18-02-14屏幕截图.png
The start position is mapped onto the smoothed reference path, which starts from the first red point you choose.
2020-05-18 18-09-32屏幕截图.png
I'll add this in Readme. Thanks for testing!

@k0suke-murakami
Copy link
Author

k0suke-murakami commented May 18, 2020

Thank you for the quick replay.

Still, trouble to get the result.
Here output with reference red points.
Screenshot from 2020-05-18 19-22-17
Screenshot from 2020-05-18 19-23-11

ref size 10
-30.5583 7.01123 0
-26.663 2.51858 0
-22.3547 0.61618 0
-18.3866 0.332707 0
-14.0783 0.0249356 0
-9.68907 -0.744231 0
-5.40515 0.202662 0
-0.894485 1.13335 0
5.82705 -0.485839 0
9.56043 -5.65033 0

@k0suke-murakami
Copy link
Author

Just let you know that it works when changing the smoothing method to TensionSmoother
Screenshot from 2020-05-18 19-48-03

Screenshot from 2020-05-18 19-49-46

@LiJiangnanBit
Copy link
Owner

I tried the input that you provided, but strangely, the failure didn't appear. 😅
2020-05-18 20-35-57屏幕截图.png
So I can't say what's wrong with AngleDiffSmoother. But it will be abandoned soon because it's too slow to solve this non-linear program. And there's also a problem you may have observed in TensionSmoother: the result has minor trembling sometimes. It's very rare, but annoying. I'm developing a new smoothing method in another branch to replace AngleDiffSmoother and TensionSmoother.

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

No branches or pull requests

2 participants