Skip to content

feat: enable Basic var and List var to coexist for the local search#1606

Merged
triceo merged 18 commits into
TimefoldAI:mainfrom
zepfred:feat/ls-mixed-model
May 30, 2025
Merged

feat: enable Basic var and List var to coexist for the local search#1606
triceo merged 18 commits into
TimefoldAI:mainfrom
zepfred:feat/ls-mixed-model

Conversation

@zepfred

@zepfred zepfred commented May 23, 2025

Copy link
Copy Markdown
Contributor

This PR completes the feature and enables the mixed model for local search.

@zepfred

zepfred commented May 23, 2025

Copy link
Copy Markdown
Contributor Author

@triceo I plan to add documentation about the mixed model once I finish the quick-start change, as I'll have concrete models to use in the explanation.

@triceo triceo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This continues to be suspiciously simple. I am really surprised.

Did you test what happens when strength/difficulty comparisons are used? Did you check entity/value sorter manner? We do not support some of it with list variables, and we need to make sure that this does not change here; and the use cases which work still need to work in the mixed mode.

@zepfred

zepfred commented May 26, 2025

Copy link
Copy Markdown
Contributor Author

This continues to be suspiciously simple. I am really surprised.

Did you test what happens when strength/difficulty comparisons are used? Did you check entity/value sorter manner? We do not support some of it with list variables, and we need to make sure that this does not change here; and the use cases which work still need to work in the mixed mode.

If there is coverage for the mentioned features, then we know the default behavior remains unchanged. I'll ensure to add tests for the relevant features.

@zepfred zepfred force-pushed the feat/ls-mixed-model branch from 1a78122 to 06da606 Compare May 26, 2025 11:50
@zepfred zepfred force-pushed the feat/ls-mixed-model branch from 165e5c2 to 8223d2c Compare May 26, 2025 17:13
@zepfred zepfred force-pushed the feat/ls-mixed-model branch from 5c951a8 to 185e79c Compare May 27, 2025 18:02
@zepfred zepfred force-pushed the feat/ls-mixed-model branch from e4ba841 to 014d677 Compare May 29, 2025 18:09

@triceo triceo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM after comments resolved.

@zepfred zepfred had a problem deploying to documentation (preview) May 30, 2025 11:27 — with GitHub Actions Failure
@zepfred zepfred force-pushed the feat/ls-mixed-model branch from ef2e929 to 11af3d6 Compare May 30, 2025 11:27
@zepfred zepfred had a problem deploying to documentation (preview) May 30, 2025 11:28 — with GitHub Actions Failure
@zepfred zepfred had a problem deploying to documentation (preview) May 30, 2025 11:37 — with GitHub Actions Failure
@sonarqubecloud

Copy link
Copy Markdown

@triceo triceo merged commit 999d2bf into TimefoldAI:main May 30, 2025
41 of 43 checks passed
@triceo triceo added this to the v1.23.0 milestone May 30, 2025
@zepfred zepfred deleted the feat/ls-mixed-model branch March 10, 2026 16:47
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.

2 participants