Repository navigation
[Temporary] Unblock BCal Luna evaluation - #958
Closed
martinsrui-msft wants to merge 8 commits into
Closed
martinsrui-msft wants to merge 8 commits into
martinsrui-msft wants to merge 8 commits into
Conversation
Contributor
Code Coverage OverviewLanguages: Python Python / code-coverage/pytestThe overall line coverage in commit eb06f2b in the Show a line coverage summary of the most impacted files.
Updated |
| from bcbench.config import get_config | ||
| from bcbench.dataset import BaseDatasetEntry, BugFixEntry, NL2ALEntry | ||
| from bcbench.evaluate.base import AgentRunner, EvaluationPipeline | ||
| from bcbench.evaluate.base import AgentRunner, EvaluationOutcome, EvaluationPipeline |
Copilot started work to resolve merge conflicts on behalf of
martinsrui-msft
October 8, 2026 09:11
View session
Copilot started work to resolve merge conflicts on behalf of
martinsrui-msft
October 8, 2026 09:23
View session
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
martinsrui-msft
force-pushed
the
martinsrui-temporary-bcal-luna-run
branch
from
October 8, 2026 12:12
875bc2e to
1bbe3e7
Compare
Allow one failed NL2AL shard to be rerun while reusing a prior attempt's complete artifacts for final completeness checks and scoring. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Let each workflow caller grant only the permissions it needs so BCal recovery can read prior artifacts without invalidating normal callers. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DO NOT MERGE
This is a temporary, run-only integration branch to unblock BCal NL2AL evaluations with the currently pinned
bc-eval[capi]==0.3.14. It must not be merged, used as a base, or used to modify native stack #945 or any existing PR branch/base.Included fixes
7df241bde459b79211acf953b48a47d1c5291171— adapt outgoing LM Checklist requests for Responses-compatible assistantoutput_textcontent and omit Lunatemperature.d05f8ca450818b69e37d1e873d86fd35702ae493— make BCal aggregation/completeness resilient so summaries and artifact handling continue after matrix failures.55a8f41b8c9d0f9fa1078e2f3a2300dedd7caf92— parse nested Luna Responses output, translate supported parameters, and fail explicitly when assistant text is absent.95503da7810f1e4a83eb72cea19b30adf9bc390f— persist/export timeout rows, keep timed-out matrix jobs failed, score timeouts deterministically withtest_passed=0, and allow complete results to reach Kusto.373008d0ee4d11c686304bacb5465d73586d8f6e— make the two existing optional-value truthiness checks explicit for the Ty version used by CI.02b00632c927cb2a6e61b03c83a3fbb48f4b1e12— persist classified CAPI HTTP 5xx failures as BCal infrastructure-error rows and upload them to Kusto without a model-quality score.1bbe3e76d44bbf44cdd6923f6d69899a0d4f68a2— adapt the temporary fixes to current main's simplified BCal API and stricter Ty checks.65c56878fb15916876b165f266e7822aef1b29bd— add a one-entry recovery mode that reuses a prior run attempt's artifacts and still requires exact 110/110 completeness before scoring or upload.eb06f2b332534c328aa0151a3e485695f9d9f718— inherit reusable-summary permissions from each caller so only BCal recovery receivesactions: readand normal evaluation callers remain valid.CAPI infrastructure-error behavior
test_passed; timeout rows keeptest_passed=0; the infrastructure row usesCoreScore=nulland preserves the error in Kusto'sErrorfield.bc-evalchanges, model changes, or dependency changes are included.One-entry recovery
Recovery mode is all-or-nothing:
recovery-source-run-id,recovery-source-run-attempt, andrecovery-entrymust all be supplied, andtest-runmust befalse. It verifies the entry belongs to the full NL2AL dataset, runs only that entry, downloads the prior attempt's evaluation artifacts into a separate subtree, and blocks scoring/upload unless the merged set is exactly 110 unique results.For run
37756156048, use:external-model:gpt-55-chat-2026-04-29test-run:falserecovery-source-run-id:37756156048recovery-source-run-attempt:1recovery-entry:nl2al__employee-emergency-contact-phone-1Temporary scope
Upstream
bc-evalwork is intentionally not consumed here. The durable implementation will be redone after the M365 LLM API uptake lands inbc-eval; this branch should be discarded after the evaluation run.Validation
bc-eval==0.3.14CLI accepted the infrastructure row with null scores and no judge call; its Kusto conversion produced normalCoreScore=1.0, infrastructureCoreScore=null, and preservedError.$/.github/...andenvironment.deploymentsyntax.git diff --checkpassed independently after push.eb06f2b332534c328aa0151a3e485695f9d9f718— passed.37756156048attempt1was verified to contain 109 unique, unexpired evaluation artifacts, no duplicate IDs, and no artifact for the recovery entry; dataset identities are unchanged on the recovery head.git diff --checkpassed.bc-eval[capi]==0.3.14.AB#653421