Skip to content

Conversation

sayanshaw24
Copy link
Collaborator

@sayanshaw24 sayanshaw24 commented Oct 15, 2025

Description

This PR updates the CI pipelines and development dependencies to align with current PyTorch and ONNX Runtime compatibility across all supported platforms.

  • CI Matrix Update:
    Dropped Python ≤ 3.10 to match ONNX Runtime’s supported versions.

  • Platform-Specific PyTorch Alignment:
    Fixed torch version (2.4.x is the last version where all unit tests - GPT-2, Whisper, etc. - still align cleanly, e.g.torch.onnx.export usage is different in latest versions) and addresses macOS wheel availability issues for newer versions (2.4.x not yet published).

    Windows & Linux:

    torch==2.4.1
    torchvision==0.19.1
    torchaudio==2.4.1
    

    macOS:

    torch==2.2.2
    torchvision==0.17.2
    torchaudio==2.2.2
    
  • Missing onnxscript dependency:
    Some tests rely on onnxscript, which was not previously listed in dev requirements, leading to ModuleNotFoundError failures in unrelated files during CI. Added onnxscript==0.1.0 to requirements-dev.txt to fix this.

  • Pinned package versions:
    Pinned the following for transient dependency mismatches and stabilized CI installs.:

    onnx==1.17.0  
    protobuf==4.25.8  
    onnxscript==0.2.7
    
  • Old Gradle Version:
    Updated Gradle wrapper validation to use v2 action.

@sayanshaw24 sayanshaw24 marked this pull request as ready for review October 15, 2025 19:05
@sayanshaw24 sayanshaw24 requested a review from a team as a code owner October 15, 2025 19:05
nenad1002
nenad1002 previously approved these changes Oct 15, 2025
@sayanshaw24 sayanshaw24 enabled auto-merge (squash) October 15, 2025 19:09
@sayanshaw24 sayanshaw24 merged commit 2b60af5 into main Oct 21, 2025
37 checks passed
@sayanshaw24 sayanshaw24 deleted the sayanshaw/fix-ci branch October 21, 2025 17:03
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.

3 participants