Skip to content

Conversation

yoyoIcy
Copy link
Contributor

@yoyoIcy yoyoIcy commented Sep 9, 2023

@paddle-bot
Copy link

paddle-bot bot commented Sep 9, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Sep 9, 2023
Tensor(shape=[3, 4], dtype=float32, place=Place(gpu:0), stop_gradient=True,
[[0.83438963, 0.70008713, 0. , 0.88831252],
[0.02200012, 0. , 0.75432241, 0.65136462],
[0.96088767, 0.82938021, 0.35367414, 0.86653489]])
Copy link
Member

Choose a reason for hiding this comment

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

softmax 的可以试试本地结果和 CI 是不是一样的

image

如果不一致的话,可以 skip 掉(仅 skip 输出部分)

>>> # doctest: +SKIP
out....
>>> # doctest: -SKIP

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这里想问下CI 给的结果是cpu 的,但原本给的结果是Place(gpu:0)的。所以是应该按照本地和CI跑的结果改输出,还是应该在开头加 >>> # doctest: +REQUIRES(env:GPU)

Copy link
Member

Choose a reason for hiding this comment

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

如非仅 GPU 能跑就不要加 env GPU 了

如果输出这么依赖于环境的话,可以考虑 skip 掉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

那我把原本给的gpu输出示例像下面这样加上skip即可?

>>> print(x)

>>> # doctest: +SKIP
Tensor(shape=[3, 4], dtype=float32, place=Place(cpu), stop_gradient=True,
        [[0.        , 0.95717543, 0.43864486, 0.        ],
         [0.84765935, 0.45680618, 0.39412445, 0.        ],
         [0.59444654, 0.        , 0.78364515, 0.        ]])
>>> # doctest: -SKIP

Copy link
Member

Choose a reason for hiding this comment

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

可以的

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow 🐾

@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Sep 11, 2023
@luotao1 luotao1 merged commit a165fa2 into PaddlePaddle:develop Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants