Skip to content

Publish and Verify FAIL_TO_PASS tests during validation - #6

Merged
Sun Haoran (haoranpb) merged 24 commits into
mainfrom
features/run-tests-during-dataset-validation
Oct 2, 2025
Merged

Publish and Verify FAIL_TO_PASS tests during validation#6
Sun Haoran (haoranpb) merged 24 commits into
mainfrom
features/run-tests-during-dataset-validation

Conversation

@haoranpb

@haoranpb Sun Haoran (haoranpb) commented Oct 2, 2025

Copy link
Copy Markdown
Collaborator

The PR contains many changes:

  • Functions to apply patches
  • Compile and Publish apps
  • Run Tests
  • Ability to run locally
  • Include Dev Setup for the container

@haoranpb Sun Haoran (haoranpb) changed the title Verify FAIL_TO_PASS tests during validation Publish and Verify FAIL_TO_PASS tests during validation Oct 2, 2025
@haoranpb
Sun Haoran (haoranpb) merged commit dba01d2 into main Oct 2, 2025
7 checks passed
@haoranpb
Sun Haoran (haoranpb) deleted the features/run-tests-during-dataset-validation branch October 2, 2025 15:42
Marko Aleksandric (AleksandricMarko) added a commit that referenced this pull request Jul 31, 2026
…ding

Applies Haoran's review feedback for the extensibility-request categories:

- Rename to explicit `extensibility-request-implement` / `extensibility-request-triage`
  across category ids, enum members, classes, dataset files and module files (#1/#2/#4).
- Slim both prompt templates to reflect real-world usage; category-specific rules now
  live in the ai-ext-fix / argus-triage skills (#3).
- Merge duplicate `bcbench.results` import in commands/evaluate.py (#5).
- Uptake #761: both ext entries subclass `RepoGroundedEntry` (#6/#7).
- Triage `patch` defaults to None; stripped from the dataset (#8).
- Constrain triage label fields with a `ManagedLabel` Literal to catch dataset typos (#9).
- Move the ext dataset classes into dataset/extensibility_request.py, like code review (#10).
- Drop `fetch_commit_if_missing` from both ext pipelines to match the other pipelines (#11).
- Keep ext-implement judge-only for now; container build/publish noted as future work (#12).
- Grade triage with the NL2AL LMChecklist instead of the code-review-style hybrid:
  `ExtRequestTriageResult` removed in favour of `JudgeBasedEvaluationResult`, expected_*
  fields replaced by an `expected` checklist, evaluators `lm_checklist`, core score
  `test_passed`; 17 triage entries migrated (#13/#14/#15/#16).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: fb712366-b026-470c-9d09-1bd9e9f52a31
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