Skip to content

[DRAFT PR] Disable ORT_RUN_EXTERNAL_ONNX_TESTS to test CI impact #25798

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kobby-kobbs
Copy link
Contributor

Description

This draft PR disables the ORT_RUN_EXTERNAL_ONNX_TESTS macro to test the CI impact of removing private model dependencies.

Changes

  • Commented out add_definitions(-DORT_RUN_EXTERNAL_ONNX_TESTS) in cmake/external/onnxruntime_external_deps.cmake (line 601)

Purpose

Testing how CI pipelines react to disabling external model tests that depend on private models not available to external partners.

Context

Part of the initiative to ensure all ONNX Runtime test models are publicly accessible for external EP partners (NVIDIA, Intel, Qualcomm, AMD).

Verification Results

  • 49.2% of current test models have verified public checksums matching ONNX Model Zoo
  • 50.8% are private/modified versions that cannot be shared
  • This change disables tests for the private models

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