Skip to content

Conversation

essos-bot
Copy link
Contributor

初步提交

任务描述

@CLAassistant
Copy link

CLAassistant commented Dec 15, 2023

CLA assistant check
All committers have signed the CLA.

)
from paddlemix.models.blip2.eva_vit import VisionTransformer
from paddlemix.models.blip2.modeling import BLIP_2_PRETRAINED_MODEL_ARCHIVE_LIST
from paddlemix.models.blip2.Qformer import BertLMHeadModel
Copy link
Collaborator

Choose a reason for hiding this comment

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

去掉无关的import


def test_save_load(self):
pass

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

from paddlemix.models.blip2.modeling import BLIP_2_PRETRAINED_MODEL_ARCHIVE_LIST
from paddlemix.models.blip2.Qformer import BertLMHeadModel
from tests.models.test_configuration_common import ConfigTester
from tests.models.test_modeling_common import (
Copy link
Collaborator

Choose a reason for hiding this comment

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

删除无关import


def test_save_load(self):
pass

Copy link
Collaborator

Choose a reason for hiding this comment

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

这个单测,跟blip2没关系吧,删掉相关代码

self.assertIsNotNone(config)


#todo @slow
Copy link
Collaborator

Choose a reason for hiding this comment

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

增加输出结果比较


create_test(name="test_dygraph", static_mode=False)
if _run_slow_test:
create_test(name="test_static", static_mode=True)
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个没有静态图推理,删掉


self.assertIsNotNone(video_frames)

setattr(OpenSetDetSamAppSlowTest, name, test_openset_det_sam)
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个跟opensetdetsam没有关系,删掉

def setUpClass(cls):
pass


Copy link
Collaborator

Choose a reason for hiding this comment

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

删掉

result = task(prompt=prompt)['result']

self.assertIsNotNone(result)

Copy link
Collaborator

Choose a reason for hiding this comment

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

增加结果对比

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个是文生图,对于生成的结果图片怎么对比呢,是预先存一个结果图片,然后比较结果图片的像素值么

Copy link
Collaborator

Choose a reason for hiding this comment

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

预存结果图片,取部分patch比较即可。

Copy link
Collaborator

Choose a reason for hiding this comment

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

无需上传



class OpenSetDetSamAppSlowTest(unittest.TestCase):
@classmethod
Copy link
Collaborator

Choose a reason for hiding this comment

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

类命名要与单测相关

def setUpClass(cls):
pass


Copy link
Collaborator

Choose a reason for hiding this comment

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

删掉


self.assertIsNotNone(result)

setattr(OpenSetDetSamAppSlowTest, name, test_openset_det_sam)
Copy link
Collaborator

Choose a reason for hiding this comment

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

删掉

create_test(name="test_dygraph", static_mode=False)
if _run_slow_test:
create_test(name="test_static", static_mode=True)

Copy link
Collaborator

Choose a reason for hiding this comment

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

没有静态图推理

Copy link
Collaborator

Choose a reason for hiding this comment

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

无需上传

Copy link
Collaborator

Choose a reason for hiding this comment

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

与该任务无关,删掉

@LokeZhou
Copy link
Collaborator

所有单测,增加结果输出对比。结果图片可以上传百度云,然后分享一下链接。

@LokeZhou
Copy link
Collaborator

LokeZhou commented Dec 22, 2023

LGTM + 需要补充结果图片

@essos-bot
Copy link
Contributor Author

test_audio_chat.py
image

@essos-bot
Copy link
Contributor Author

test_audio-to-Caption.py

image

@essos-bot
Copy link
Contributor Author

test_autolabel.py

image

@essos-bot
Copy link
Contributor Author

test_DualTextAndImageGuidedGeneration.py
image

@essos-bot
Copy link
Contributor Author

test_Image2ImageTextGuidedGeneration.py
image

@essos-bot
Copy link
Contributor Author

test_MusicGeneration

failed原因:暂时没有minigpt4权重
image

@essos-bot
Copy link
Contributor Author

test_text2image.py
image

@essos-bot
Copy link
Contributor Author

test_text2video.py

image

@essos-bot
Copy link
Contributor Author

test_TextGuidedImageInpainting.py

image

@essos-bot
Copy link
Contributor Author

test_TextGuidedImageUpscaling.py

image

@essos-bot
Copy link
Contributor Author

test_groundingdino.py
image

@essos-bot
Copy link
Contributor Author

essos-bot commented Dec 26, 2023

test_sam.py

image

@essos-bot
Copy link
Contributor Author

补充遗漏的三个,改用线上资源地址测试通过:

image image image

@LokeZhou
Copy link
Collaborator

基本符合要求,可以合入

@LokeZhou LokeZhou merged commit 3a3f405 into PaddlePaddle:develop Dec 27, 2023
westfish pushed a commit to westfish/PaddleMIX that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants