Repository navigation
[Temporary] Unblock BCal Luna evaluation - #958
Draft
martinsrui-msft wants to merge 6 commits into
Draft
martinsrui-msft wants to merge 6 commits into
martinsrui-msft wants to merge 6 commits into
Conversation
Contributor
Code Coverage OverviewLanguages: Python Python / code-coverage/pytestThe overall line coverage in commit 1bbe3e7 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
This branch has not been deployed
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.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.Temporary 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.bc-eval[capi]==0.3.14.AB#653421