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

【论文复现144】Seaformer #3011

Merged
merged 39 commits into from
Jun 1, 2023

Conversation

marshall-dteach
Copy link
Contributor

@marshall-dteach marshall-dteach commented Feb 28, 2023

PR types

PR changes

Description

由于竞争的原因 快速pr 占坑, 代码写的比较乱 后面还会慢慢修改(可以先不用review)

@marshall-dteach
Copy link
Contributor Author

image
目前这个参数比较复杂 不知道以什么形式配置

Copy link
Contributor

@zhaoyantao-murray zhaoyantao-murray left a comment

Choose a reason for hiding this comment

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

明天上午我再接着review哈。

configs/seaformer/seaformer_ade20k_512x512_160k.yml Outdated Show resolved Hide resolved
configs/seaformer/seaformer_ade20k_512x512_160k.yml Outdated Show resolved Hide resolved
paddleseg/models/__init__.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
Copy link
Contributor

@zhaoyantao-murray zhaoyantao-murray left a comment

Choose a reason for hiding this comment

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

第一次的review先这些,有空你修改一下;

paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
@WanderJN
Copy link

WanderJN commented Mar 9, 2023

您好,想问一下paddleseg在线加载预训练模型那个pretrained:参数里边的链接是在哪里查找,的一直找不到

@Sunting78 Sunting78 requested a review from juncaipeng May 12, 2023 06:04
Sunting78
Sunting78 previously approved these changes May 12, 2023
Copy link
Collaborator

@Sunting78 Sunting78 left a comment

Choose a reason for hiding this comment

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

lgtm

configs/seaformer/README.md Outdated Show resolved Hide resolved
paddleseg/models/seaformer.py Outdated Show resolved Hide resolved
@Sunting78 Sunting78 self-requested a review May 12, 2023 06:28
@Sunting78 Sunting78 dismissed their stale review May 18, 2023 06:40

等待修改

warmup_iters: 1500
warmup_start_lr: 1.0e-6

test_config:
Copy link
Collaborator

Choose a reason for hiding this comment

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

seaformer源码不是滑窗测试,请用相同测试方式

@@ -0,0 +1,37 @@
_base_: '../_base_/cityscapes.yml'
Copy link
Collaborator

Choose a reason for hiding this comment

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

文件名是ade20k 但用的是cityscapes.yml ? 没看明白

Copy link
Collaborator

@Sunting78 Sunting78 left a comment

Choose a reason for hiding this comment

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

无法按照slide=False复现模型精度,请将slide设置为False后评测

Copy link
Collaborator

@Sunting78 Sunting78 left a comment

Choose a reason for hiding this comment

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

val_dataset:
transforms:
- type: Resize
target_size: [2048, 512]
keep_ratio: True
size_divisor: 32
- type: Normalize
mean: [0.485, 0.456, 0.406]
std: [0.229, 0.224, 0.225]


| Model | size | Head | Resolution | Training Iters | mIoU (slice) | Links |
| :-: | :-: | :-: | :-: | :-: | :-: | :-: |
| SeaFormer | base | LightHead | 512x512 | 160000 | 40.92 | [model](https://paddleseg.bj.bcebos.com/dygraph/ade20k/seaformer_ade20k_512x512_160k/model.pdparams) \| [log](https://paddleseg.bj.bcebos.com/dygraph/ade20k/seaformer_ade20k_512x512_160k/train.log) \| [vdl](https://www.paddlepaddle.org.cn/paddle/visualdl/service/app/scalar?id=f43d0185995605fb99da6597121a96f1) |
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.

等训练完另提pr吧,格式改了很多,得调模型文件的权重名

paddleseg/models/backbones/seaformer.py Outdated Show resolved Hide resolved
paddleseg/models/seg_seaformer.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@juncaipeng juncaipeng left a comment

Choose a reason for hiding this comment

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

paddleseg/models/seaformer_seg.py Outdated Show resolved Hide resolved
paddleseg/models/seaformer_seg.py Show resolved Hide resolved
juncaipeng

This comment was marked as spam.

juncaipeng

This comment was marked as spam.

@marshall-dteach
Copy link
Contributor Author

image
静态模型测试图

@Sunting78
Copy link
Collaborator

image 静态模型测试图

换成ade20k的 输出正常结果

@marshall-dteach
Copy link
Contributor Author

image
重新上传静态模型推理图

Copy link
Collaborator

@juncaipeng juncaipeng left a comment

Choose a reason for hiding this comment

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

LGTM

@juncaipeng juncaipeng merged commit 6c9b833 into PaddlePaddle:develop Jun 1, 2023
3 checks passed
@shiyutang shiyutang added the contributor Contribution from developers label Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor Contribution from developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants