Skip to content

Cast of "max_velocity" and "max_acceleration" values to double (backport #2803)#3038

Merged
sjahr merged 1 commit intomoveit:humblefrom
mikeferguson:humble_2803
Oct 25, 2024
Merged

Cast of "max_velocity" and "max_acceleration" values to double (backport #2803)#3038
sjahr merged 1 commit intomoveit:humblefrom
mikeferguson:humble_2803

Conversation

@mikeferguson
Copy link
Copy Markdown
Contributor

Description

Cherry pick #2803 to humble

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

…t#2803)

Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
@mergify
Copy link
Copy Markdown

mergify bot commented Oct 23, 2024

Please target the main branch for development, we will backport the changes to humble for you if approved and if they don't break API.

@mikeferguson mikeferguson changed the title Cast of "max_velocity" and "max_acceleration" values to double (#2803) Cast of "max_velocity" and "max_acceleration" values to double (backport #2803) Oct 23, 2024
@sjahr sjahr merged commit f6b82c6 into moveit:humble Oct 25, 2024
@mikeferguson mikeferguson deleted the humble_2803 branch October 25, 2024 09:58
@Pugens
Copy link
Copy Markdown

Pugens commented Feb 26, 2025

Issue is still present on jazzy

@mikeferguson
Copy link
Copy Markdown
Contributor Author

Issue is still present on jazzy

Not sure what you're seeing - but this patch is definitely in the Jazzy branch (since Jazzy was not cut until after the original PR was merged). I'd suggest to open a new issue with the details of whatever issue you're running into.

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.

4 participants