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

【开源实习】Mask2Former模型应用开发 #1770

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

linrx-ctrl
Copy link
Contributor

	renamed:    tutorials/AST/Audio Spectrogram Transformer.ipynb -> applications/AST/Audio Spectrogram Transformer.ipynb
	renamed:    applications/Inference_with_Mask2Former.ipynb -> applications/Mask2Former/Inference_with_Mask2Former.ipynb
@linrx-ctrl
Copy link
Contributor Author

AST的文件夹从tutorials调整到applications
https://gitee.com/mindspore/community/issues/IAAD7O

@linrx-ctrl
Copy link
Contributor Author

fix bug about mask2former

1 similar comment
@linrx-ctrl
Copy link
Contributor Author

fix bug about mask2former

@linrx-ctrl
Copy link
Contributor Author

@cui0523
Copy link
Contributor

cui0523 commented Nov 4, 2024

截图部分代码运行结果与原应用执行结果误差较大,排查迁移代码是否正确

20241104-154505(WeLinkPC)
20241104-154511(WeLinkPC)

@linrx-ctrl
Copy link
Contributor Author

经排查,是迁移代码中mask2former的image_processing_mask2former.py中一处小错误导致的

通过在函数部分,输出中间变量:(发现错误值)
以下是mindnlp的截图,其中label变量的输出(错误)
496f59b2b0e3f0c4da9728e4ccb77462
9a3409c9a3104f6329b23da74fd24b74

以下是transformer的截图,其中label变量的输出(正确)
image
image

@linrx-ctrl
Copy link
Contributor Author

@linrx-ctrl
Copy link
Contributor Author

b4ffd448241ccb890122ed8ab6974c52

@cui0523
Copy link
Contributor

cui0523 commented Nov 14, 2024

验收通过,可以合入代码

@lvyufeng
Copy link
Collaborator

pylint没过

@linrx-ctrl
Copy link
Contributor Author

修改后通过了pylint

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