Skip to content

[common-artifacts] Exclude Inf_Squeeze_000 recipe#14864

Merged
seanshpark merged 1 commit intoSamsung:masterfrom
mbencer:mbencer/ExcludeSqueeze
Mar 27, 2025
Merged

[common-artifacts] Exclude Inf_Squeeze_000 recipe#14864
seanshpark merged 1 commit intoSamsung:masterfrom
mbencer:mbencer/ExcludeSqueeze

Conversation

@mbencer
Copy link
Copy Markdown
Contributor

@mbencer mbencer commented Mar 21, 2025

This commit excludes Inf_Squeeze_000 because TestDataGenerator does not support unknown dimensions.

ONE-DCO-1.0-Signed-off-by: Mateusz Bencer m.bencer@partner.samsung.com

Draft: #14882

@seanshpark
Copy link
Copy Markdown
Contributor

can you plz add links to issue and draft PR if exist?

This commit excludes Inf_Squeeze_000 because TestDataGenerator does not support unknown dimensions.

ONE-DCO-1.0-Signed-off-by: Mateusz Bencer <m.bencer@partner.samsung.com>
@mbencer mbencer force-pushed the mbencer/ExcludeSqueeze branch from 727f9c0 to ba7e6b3 Compare March 24, 2025 08:43
@mbencer mbencer requested a review from seanshpark March 26, 2025 08:02
@seanshpark
Copy link
Copy Markdown
Contributor

@mbencer , you are adding Inf_Squeeze_000 but this PR title and commit message is from the old one.
please double check with out PRs too.

@mbencer mbencer changed the title [common-artifacts] Exclude Squeeze_dynamic_000 recipe [common-artifacts] Exclude Inf_Squeeze_000 recipe Mar 26, 2025
@mbencer
Copy link
Copy Markdown
Contributor Author

mbencer commented Mar 26, 2025

@mbencer , you are adding Inf_Squeeze_000 but this PR title and commit message is from the old one. please double check with out PRs too.

@seanshpark right - I've updated only commit description, thank for catching it ;-)

@seanshpark
Copy link
Copy Markdown
Contributor

I'll do the approval after someone in one_compiler has approved.

@mbencer
Copy link
Copy Markdown
Contributor Author

mbencer commented Mar 26, 2025

@Samsung/one_compiler Could you please take a look? ;-)

Copy link
Copy Markdown
Collaborator

@jinevening jinevening left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit 90c030a into Samsung:master Mar 27, 2025
7 checks passed
@mbencer mbencer deleted the mbencer/ExcludeSqueeze branch May 2, 2025 08:12
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