[Contribs] Add CPS#3075
Merged
Merged
Conversation
Contributor
Author
|
@Bobholamovic 晖哥有空的时候麻烦review一下 |
Bobholamovic
requested changes
Mar 30, 2023
Contributor
Author
好的👌，辛苦晖哥~ |
Bobholamovic
requested changes
Apr 4, 2023
Member
Bobholamovic
left a comment
Bobholamovic
left a comment
There was a problem hiding this comment.
留了一些意见，另外请留意一下我对上一次review留下的conversation的回复~
| """ | ||
| Configuration parsing. | ||
|
|
||
| The following hyper-parameters are available in the config file: |
Bobholamovic
requested changes
Apr 4, 2023
Member
Bobholamovic
left a comment
Bobholamovic
left a comment
There was a problem hiding this comment.
留了一些意见，另外请留意一下我对上一次review留下的conversation的回复~
Contributor
Author
辛苦晖哥了~ 我大致看了一下 先把能改的改了 |
Bobholamovic
requested changes
Apr 17, 2023
Member
Bobholamovic
left a comment
Bobholamovic
left a comment
There was a problem hiding this comment.
辛苦修改！我注意到上一轮review中的一些意见标注了done，但好像还没有得到修改（我在本轮review中新开了一些conversation做提醒，可以不用翻找之前的review了）~ 另外，也留了一些新意见。
Member
Bobholamovic
left a comment
Bobholamovic
left a comment
There was a problem hiding this comment.
上次review的conversation中有两处好像还没有解决
|
|
||
| def uniform_(tensor, a, b): | ||
| """ | ||
| Modify tensor in space using uniform_ |
|
|
||
| def uniform_(tensor, a, b): | ||
| """ | ||
| Modify tensor in space using uniform_ |
| Configuration parsing. | ||
|
|
||
| The following hyper-parameters are available in the config file: | ||
| batch_size: The number of samples total gpus. |
juncaipeng
reviewed
Apr 20, 2023
|
|
||
| ## 环境配置 | ||
|
|
||
|
|
Collaborator
There was a problem hiding this comment.
这部分环境配置，最好明确给出目前训练验证跑通的paddle版本。
Contributor
Author
There was a problem hiding this comment.
我是用脚本任务里的develop版本跑到78.39%，用2.4.0达不到这个精度，那我这里改为PaddlePaddle develop (Nightly build)版本
|
|
||
| ## 数据准备 | ||
|
|
||
| 使用CPS源代码所提供的Cityscapes数据集，下载链接为：https://pkueducn-my.sharepoint.com/:f:/g/personal/pkucxk_pku_edu_cn/EtjNKU0oVMhPkOKf9HTPlVsBIHYbACel6LSvcUeP4MXWVg?e=139icd，将数据集`city`放至`contrib/CrossPseudoSupervision/data`文件夹下，准备好的数据组织如下： |
juncaipeng
approved these changes
Apr 23, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR types
New features
PR changes
Contribs
Description
Add new contribs for CPS