Skip to content

docs: clarify CH need for Neighborhoods API#2288

Merged
triceo merged 2 commits into
TimefoldAI:mainfrom
TomCools:docs/mention-ch-in-neighborhood
May 5, 2026
Merged

docs: clarify CH need for Neighborhoods API#2288
triceo merged 2 commits into
TimefoldAI:mainfrom
TomCools:docs/mention-ch-in-neighborhood

Conversation

@TomCools
Copy link
Copy Markdown
Contributor

@TomCools TomCools commented May 5, 2026

If you have never needed the solverconfig.xml file, but want to try the Neighborhoods API, it's not entirely clear that you also need to add a CH phase when you explicitly set the localsearch phase. Tried to clarify that here.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Neighborhoods API documentation to clarify how it should be configured within the solver phase pipeline, specifically when users explicitly define a local search phase.

Changes:

  • Adds a constructionHeuristic entry to the XML solver configuration example for Neighborhoods API usage.
  • Adds a note explaining that Neighborhoods runs as part of Local Search and therefore expects an initialized solution.
  • Adjusts the surrounding docs to make the setup steps more explicit for users configuring phases manually.

Comment thread docs/src/modules/ROOT/pages/optimization-algorithms/neighborhoods.adoc Outdated
Comment thread docs/src/modules/ROOT/pages/optimization-algorithms/neighborhoods.adoc Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

@triceo triceo merged commit 1dbae9f into TimefoldAI:main May 5, 2026
14 checks passed
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