Skip to content

Conversation

tuanddd
Copy link

@tuanddd tuanddd commented Jul 31, 2025

  • What kind of change does this PR introduce? This PR updates the AxMultiMetricFn type definition so that it can support asynchronous code

  • What is the current behavior? AxMultiMetricFn only accept non-async function due to its return type Record<string, number>

  • What is the new behavior (if this is a feature change)? Added Promise<Record<string, number>> as one of the return type so user can use the metricFn as asynchronous (LLM-judge, etc...)

  • Other information:

    • Verified that all function invocations in src/ax/dsp/optimizer.ts correctly have await before

@tuanddd
Copy link
Author

tuanddd commented Jul 31, 2025

@dosco cheers

@tuanddd
Copy link
Author

tuanddd commented Aug 21, 2025

@dosco what's blocking this?

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.

1 participant