Skip to content

COPY of PR #2359#2463

Draft
shaahji wants to merge 24 commits into
mainfrom
liujij/feat-sd-models
Draft

COPY of PR #2359#2463
shaahji wants to merge 24 commits into
mainfrom
liujij/feat-sd-models

Conversation

@shaahji
Copy link
Copy Markdown
Collaborator

@shaahji shaahji commented May 13, 2026

Copy of PR #2359

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

@liujij
Copy link
Copy Markdown

liujij commented May 15, 2026

Hi @shaahji Could you please sync with PR2359 and retrigger the Olive CI? Thanks a lot!


pytest.importorskip("model_generate", reason="model_generate is not installed; skipping all SD model generation tests")

from olive.model import ONNXModelHandler
pytest.importorskip("model_generate", reason="model_generate is not installed; skipping all SD model generation tests")

from olive.model import ONNXModelHandler
from olive.passes.olive_pass import create_pass_from_dict

from olive.model import ONNXModelHandler
from olive.passes.olive_pass import create_pass_from_dict
from olive.passes.onnx.vitis_ai.vitis_generate_model_sd import VitisGenerateModelSD
from olive.model import ONNXModelHandler
from olive.passes.olive_pass import create_pass_from_dict
from olive.passes.onnx.vitis_ai.vitis_generate_model_sd import VitisGenerateModelSD
from test.utils import ONNX_MODEL_PATH, get_onnx_model
@shaahji
Copy link
Copy Markdown
Collaborator Author

shaahji commented May 19, 2026

Hi @shaahji Could you please sync with PR2359 and retrigger the Olive CI? Thanks a lot!

Hey @liujij, build is reporting a few lint error. Could you please fix them. Thanks!

@jambayk jambayk closed this May 20, 2026
@jambayk jambayk reopened this May 20, 2026
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.

4 participants