Skip to content

[CI] Re-enable select onnx_ops tests#23216

Closed
amd-eochoalo wants to merge 1 commit intoiree-org:mainfrom
amd-eochoalo:eochoa/2026-01-20/re-enable-select-tests
Closed

[CI] Re-enable select onnx_ops tests#23216
amd-eochoalo wants to merge 1 commit intoiree-org:mainfrom
amd-eochoalo:eochoa/2026-01-20/re-enable-select-tests

Conversation

@amd-eochoalo
Copy link
Contributor

These unskipped tests will fail on CI. The issue is explained here: #23189 Only:

  • test_gru_seq_length
  • test_rnn_seq_length
  • test_scan9_sum
  • test_stft_with_window

are selected for execution since they do generate dispatches and are considered bugs.

Tests:

  • test_tfidfvectorizer_tf_batch_onlybigrams_skip0
  • test_tfidfvectorizer_tf_batch_onlybigrams_skip5
  • test_tfidfvectorizer_tf_batch_uniandbigrams_skip5

Are not selected for execution since they are lower priority. They do not generate dispatches. These tests all started failing with #22813

@amd-eochoalo amd-eochoalo changed the title [CI] Re-enable select tests [CI] Re-enable select onnx_ops tests Jan 20, 2026
@amd-eochoalo
Copy link
Contributor Author

@kuhar thanks, I'll wait for @MaheshRavishankar input here since he wanted to go over why these tests are failing.

Copy link
Collaborator

@MaheshRavishankar MaheshRavishankar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we see the input IR for these tests. Its not available on the issue. For now lets leave those out. In the tagged issue if we add the input IR Ben will be able to fix it and he can re-enable then.

@amd-eochoalo
Copy link
Contributor Author

@MaheshRavishankar can you look into the ticket? I've posted files with the full IR after every pass and a focus also on the IR after AutoInputConversionPipelinePass

@amd-eochoalo
Copy link
Contributor Author

I'm going to close this branch as #23189 clarifies this and has already been assigned.

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