Skip to content

December 2024 update #79

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

Merged
merged 40 commits into from
Feb 6, 2025
Merged

Conversation

johnwason
Copy link
Contributor

No description provided.

@johnwason
Copy link
Contributor Author

johnwason commented Nov 30, 2024

The OMPL test is failing on MacOS for Python 3.13 on both x64 and arm64. Puzzlingly the test works correctly with Python 3.12 on both MacOS x64 and arm64.

The error is:

Error:   OMPLPlanner failed to generate problem: In OMPLDefaultPlanProfile: All goal states are either in collision or outside limits.
tests/tesseract_motion_planning/test_ompl_planner.py F
         at line 128 in /Users/runner/work/tesseract_python/tesseract_python/ws/src/tesseract_planning/tesseract_motion_planners/ompl/src/ompl_motion_planner.cpp

This PR is ready since this exact failure isn't a critical problem.

@johnwason johnwason marked this pull request as ready for review November 30, 2024 21:59
@johnwason
Copy link
Contributor Author

@Levi-Armstrong I need release tags at these commit:

  • tesseract: c0162ef9a30507d0ef750d37aa96d098846a4e38
  • tesseract_planning: b8f02cc8382eb86b1a77a2369acccf01f646499a
  • trajopt: 0840fc737e2fcc4441fa50d036467c5f17f6746b

@xbkaishui
Copy link

Hello, this is a big update. when is can be merge into master? thanks

@johnwason
Copy link
Contributor Author

@xbkaishui some time this week most likely

@xbkaishui
Copy link

@xbkaishui some time this week most likely

Great, Thanks

@Levi-Armstrong
Copy link
Contributor

I am in the process of creating release 0.28.X which would be a good tag to leverage.

@johnwason
Copy link
Contributor Author

@Levi-Armstrong the 0.28.X versions appear to work. In the future let me know before you make a release so I can make sure there aren't any issues on Windows and OSX.

@Levi-Armstrong
Copy link
Contributor

Let me know if there are any issues and I can create a patch release

@johnwason
Copy link
Contributor Author

@Levi-Armstrong I have trajopt_ifopt building on Windows without too much difficulty but I will need to patch some of the cmake scripts in the ifopt package.

@Levi-Armstrong
Copy link
Contributor

If you create a PR I think I have the ability to get it merged.

@johnwason
Copy link
Contributor Author

I have not had a chance to finalize these updates and push to pypi. If you need the updated wheels sooner, you can pull them from the most recent artifacts from the CI runs here: https://github.com/tesseract-robotics/tesseract_python/actions/runs/12818245676

The wheel will be in the wheelhouse directory. You can install from the *.whl file using pip.

@johnwason
Copy link
Contributor Author

johnwason commented Feb 6, 2025

The update is done. I will merge and push to PyPi once the builds are ready.

@johnwason johnwason merged commit c2a1a03 into tesseract-robotics:master Feb 6, 2025
16 of 18 checks passed
@johnwason
Copy link
Contributor Author

Updated packages are available on the release https://github.com/tesseract-robotics/tesseract_python/releases/tag/0.5.1 and on PyPi for installation using pip.

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.

3 participants