Skip to content

Add surrogate free indexed strategies - #50

Open
vreshniak wants to merge 7 commits into
developfrom
indexed_strategies
Open

Add surrogate free indexed strategies#50
vreshniak wants to merge 7 commits into
developfrom
indexed_strategies

Conversation

@vreshniak

@vreshniak vreshniak commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Add Indexed Strategies Support and Improve Strategy Handling

Summary

This PR introduces indexed strategies (surrogate-free sampling methods) and refactors strategy handling throughout the DIAL service.

Features

Indexed Strategies

Adds new surrogate-free sampling strategies that don't require model training:

  • center: Sample at domain center point
  • corners: Sample at domain corner points
  • grid: Uniform grid-based sampling
  • chebyshev: Chebyshev node-based sampling
  • latin_hypercube: Latin hypercube design sampling

New Examples

  • Added 2d_rosenbrock_client_with_strategies.py demonstrating available strategies

Changes Overview

Files Modified
File Changes
src/dial_service/utilities/strategies.py Core indexed strategies implementation
src/dial_dataclass/dial_dataclass.py Updated strategy parameter definitions
src/dial_service/dial_service.py Service-level strategy validation
src/dial_service/core.py Strategy execution logic
scripts/2d_rosenbrock_client_with_strategies.py New example script

@vreshniak
vreshniak force-pushed the indexed_strategies branch from f867119 to 0f86d61 Compare July 22, 2026 17:50
@Lance-Drane

Copy link
Copy Markdown
Contributor

Could you add tests for the new strategies? Unit tests alone should be fine. You can either use the existing file in tests/unit/ or create your own file(s) in tests/unit (as long as the tests are in tests/unit/ I am impartial as to how they are organized)

API looks fine

Signed-off-by: Lance-Drane <Lance-Drane@users.noreply.github.com>
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