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

noise.sdf: add min="0.0" to the precision and dynamic_bias_correlation_time parameters #231

Open
osrf-migration opened this issue Apr 10, 2020 · 1 comment
Labels
bug Something isn't working minor

Comments

@osrf-migration
Copy link

Original report (archived issue) by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


This is a follow-up task from pr #329, which proposed to "Add random walk params to noise for sdf 1.6". Most of the changes in this PR have already been merged in PR #525 except for the differences noted in the following comment:

Two of the differences are still worth applying, but I'm putting them in an issue to be resolved after the migration to GitHub.

  • noise.sdf: add min="0.0" to the precision parameter
  • noise.sdf: add min="0.0" to the dynamic_bias_correlation_time parameter
@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


For reference, the changes from pr #329 have been pulled into this repo as 9b5a4bc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working minor
Projects
None yet
Development

No branches or pull requests

3 participants