Skip to content
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

[FIX] fix OPs are not supported #744 #1092

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

masteryi-0018
Copy link

Now it's only a simple implementation to solve the problem #744, please optimize it together.

@CLAassistant
Copy link

CLAassistant commented Oct 31, 2024

CLA assistant check
All committers have signed the CLA.

@luotao1
Copy link
Collaborator

luotao1 commented Nov 1, 2024

  1. 请通过Code-Style流水线
  2. 请在autoscantest流水线补充单侧:https://github.com/PaddlePaddle/X2Paddle/tree/develop/test_autoscan/torch

@luotao1 luotao1 self-assigned this Nov 1, 2024
@luotao1 luotao1 added the contributor External developers label Nov 1, 2024
@masteryi-0018
Copy link
Author

As CI mentioned Details, when I run:

pre-commit run --files x2paddle/core/program.py x2paddle/op_mapper/onnx2paddle/opset12.py

an error occured:

An unexpected error has occurred: CalledProcessError: command: ('/usr/bin/git', 'fetch', 'origin', '--tags')
return code: 128
expected return code: 0
stdout: (none)
stderr:
    remote: Repository not found.
    fatal: repository 'https://github.com/PaddlePaddle/mirrors-yapf.git/' not found

I have tried for several times and always get this error.

@luotao1
Copy link
Collaborator

luotao1 commented Nov 18, 2024

image

你的develop分支太老了,需要merge下最新的develop分支。

@luotao1
Copy link
Collaborator

luotao1 commented Nov 22, 2024

image 需要修复下新加的单侧

@masteryi-0018
Copy link
Author

@luotao1 please check this pull request, and correct me if there's any questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants