Skip to content

Update ompl to 2.0.1 - #64

Merged
traversaro merged 9 commits into
conda-forge:mainfrom
mini-1235:codex/ompl-2-0-1-feedstock-fix
Jul 31, 2026
Merged

Update ompl to 2.0.1#64
traversaro merged 9 commits into
conda-forge:mainfrom
mini-1235:codex/ompl-2-0-1-feedstock-fix

Conversation

@mini-1235

@mini-1235 mini-1235 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update OMPL to 2.0.1
  • remove the stale Boost 1.90 patch, which no longer applies to OMPL 2.0.1
  • update the Unix installed-header test for the OMPL 2.x include layout

This supersedes the previous bot-generated update PR: #63

Root cause

The 2.0.1 bot update still carried the feedstock patch and test path from the 1.7.x recipe. OMPL 2.0.1 already has the Boost dependency/link changes upstream, so the old patch fails during source patching. The installed headers are also no longer under include/ompl-1.7/.

Validation

  • git diff --check

@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/recipe.yaml) and found it was in an excellent condition.

@mini-1235

Copy link
Copy Markdown
Contributor Author

Related to RoboStack/ros-rolling#31, since 2.0.1 is released in ros rolling

@traversaro

Copy link
Copy Markdown
Contributor

fyi @nmarticorena as he was interested in this in #62 .

@traversaro

Copy link
Copy Markdown
Contributor

I created https://github.com/conda-forge/ompl-feedstock/tree/v1.7, can you please add:

bot:
  abi_migration_branches:
  - v1.7

to conda-forge.yml so that we still get ABI migration PRs to the v1.7 branch? Thanks!

Comment thread conda-forge.yml Outdated
Co-authored-by: Silvio Traversaro <silvio@traversaro.it>
@traversaro

Copy link
Copy Markdown
Contributor

@conda-forge-admin please rerender

@traversaro

Copy link
Copy Markdown
Contributor

@conda-forge-admin please rerender

@traversaro

Copy link
Copy Markdown
Contributor

test_pdf seems quite flaky and non-reproducible:

 │ │  6/21 Test  #6: test_pdf .........................***Failed    0.06 sec
 │ │ Running 2 test cases...
 │ │ $SRC_DIR/tests/datastructures/pdf.cpp:178: error: in "Statistical": absolute value of (sampleMean) - (mean){-0.001872600000000002} exceeds 0.0016686446595965243
 │ │ *** 1 failure is detected in the test module "PDF"
 │ │ 

perhaps it uses some non-costant value as a seed for the RNG?

@traversaro
traversaro merged commit 1562385 into conda-forge:main Jul 31, 2026
47 of 48 checks passed
@mini-1235
mini-1235 deleted the codex/ompl-2-0-1-feedstock-fix branch July 31, 2026 14:55
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