Paddle icon indicating copy to clipboard operation
Paddle copied to clipboard

【Hackathon 6th No.26】为 paddle.view 进行功能增强 -part

Open yinfan98 opened this issue 9 months ago • 11 comments

PR Category

Others

PR Types

Improvements

Description

增强paddle.view,使其能支持动态推理shape

yinfan98 avatar May 10 '24 19:05 yinfan98

你的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[bot] avatar May 10 '24 19:05 paddle-bot[bot]

https://github.com/PaddlePaddle/Paddle/pull/63134 之前的PR已实现 XavierNormal gain

yinfan98 avatar May 10 '24 20:05 yinfan98

b)view 的 shape_or_dtype 参数 不支持 paddle 原生数据类型,如 padlde.int32 作为输入。

试了一下 感觉是支持paddle.xx这种作为输入的。另外我全局搜了一下test。似乎没搜到view 的 test。请问还需要在哪里补充一下测试用例么

yinfan98 avatar May 11 '24 16:05 yinfan98

@luotao1 @zhwesky2010 hi, 我似乎没在test里找到关于view的test。我需要为-1的shape推导单独写一个test么。如果需要我该写在什么位置呢。谢谢~(我已经测试过,并确定shape推导没问题了)

yinfan98 avatar May 13 '24 14:05 yinfan98

@luotao1 @zhwesky2010 hi, 我似乎没在test里找到关于view的test。我需要为-1的shape推导单独写一个test么。如果需要我该写在什么位置呢。谢谢~(我已经测试过,并确定shape推导没问题了)

在test_stride.py里面,在这个里面加一个case吧:https://github.com/PaddlePaddle/Paddle/blob/develop/test/deprecated/legacy_test/test_stride.py#L519-L557

zhwesky2010 avatar May 14 '24 10:05 zhwesky2010

@yinfan98 看一下PR-CI-APPROVAL 的失败原因,需要使用 np.tesing.assert 来测试

zhwesky2010 avatar May 15 '24 04:05 zhwesky2010

在python api接口添加了代码示例,以便docs能同步更新~

yinfan98 avatar May 16 '24 06:05 yinfan98

静态检查没过,请问是因为我在view 的 api接口的注释里使用print了吗?这种情况下我需要修改哪里或者如何通过一下嘞

yinfan98 avatar May 17 '24 07:05 yinfan98

静态检查没过,请问是因为我在view 的 api接口的注释里使用print了吗

这个没有关系,最后review通过后,可以豁免

luotao1 avatar May 17 '24 07:05 luotao1

这个没有关系,最后review通过后,可以豁免

get~谢谢。 FYI:在昨天的基础上,添加了1. 四维到二维 2. 四维到一维 3. 四维到五维(低维到高维)的形状推导test。 请再review下我的pr吧 @zhwesky2010 @xuxinyi389 @wanghuancoder ,谢谢

yinfan98 avatar May 17 '24 08:05 yinfan98

LGTM

xuxinyi389 avatar May 17 '24 10:05 xuxinyi389