Make OMPL an optional installation #178
Conversation
There was a problem hiding this comment.
Say something like
THIS IS AN OPTIONAL FUNCTION, WON'T AFFECT YOUR USE
Unable to import OmplManagerAttr. OMPL is likely not installed: skipping.
If you haven't installed OMPL before, install it using python3 -m pip install https://github.com/ompl/ompl/releases/download/prerelease/ompl-1.6.0-cp310-cp310-manylinux_2_28_x86_64.whl
Otherwise, make sure you install the python bindings.
There was a problem hiding this comment.
Just updated the print message!
I'd like to highlight that this will print every time any executable is run, lol. Just wondering if this wall of text might confuse people? That's why I had it kinda short before. But your call!
There was a problem hiding this comment.
I see.. then I think just the first line would work.
Unable...will not affect your use
|
Also update the readme with this? Say it's optional |
|
README! |
Description:
Fixes #177.
If OMPL is not installed, a helpful error message is printed instead of crashing.
I'm not sure if we want that error message to print; perhaps there is a better UX we can provide.