Skip to content

  • 体验新版
    • 正在加载...
  • 登录
  • PaddlePaddle
  • Paddle
  • 合并请求
  • !27193

P
Paddle
  • 项目概览

PaddlePaddle / Paddle
接近 2 年 前同步成功

通知 2323
Star 20933
Fork 5424
  • 代码
    • 文件
    • 提交
    • 分支
    • Tags
    • 贡献者
    • 分支图
    • Diff
  • Issue 1423
    • 列表
    • 看板
    • 标记
    • 里程碑
  • 合并请求 543
  • Wiki 0
    • Wiki
  • 分析
    • 仓库
    • DevOps
  • 项目成员
  • Pages
P
Paddle
  • 项目概览
    • 项目概览
    • 详情
    • 发布
  • 仓库
    • 仓库
    • 文件
    • 提交
    • 分支
    • 标签
    • 贡献者
    • 分支图
    • 比较
  • Issue 1,423
    • Issue 1,423
    • 列表
    • 看板
    • 标记
    • 里程碑
  • 合并请求 543
    • 合并请求 543
  • Pages
  • 分析
    • 分析
    • 仓库分析
    • DevOps
  • Wiki 0
    • Wiki
  • 成员
    • 成员
  • 收起侧边栏
  • 动态
  • 分支图
  • 创建新Issue
  • 提交
  • Issue看板

change sequence length attribute to input !27193

  • Report abuse
!27193 已合并 9月 08, 2020 由 saxon_zh@saxon_zh 创建
#<User:0x00007ff7ffa86a98>
  • 概览 16
  • 提交 9
  • 变更 6

Created by: GaoWei8

PR types

Bug fixes

PR changes

OPs

Describe

将lstm的sequence length由参数改成输入

需要approve的list

  • const_cast的使用,没有新增 @Xreki

  • SequenceLength的shape不需要大于100 @luotao1

  • 替换attr sequence_length为输入变量SequenceLength。 相对于版本1.8.4, 新增AsDispensable(可有可无)输入变量SequenceLength, 兼容原有 api/op 的使用, 已通过class TestCUDNNlstmAPI(unittest.TestCase)进行验证。@Superjomn

不会造成接口兼容性问题的论证

  • 对attribute的修改并没有跨版本,因此只有在develop分支中新增了输入,且输入可以为空。

  • 原有lstm接口的双向在计算结果的维度会产生错误。

  • 原有lstm接口的多层的结果有问题,原有接口一直用的输入是padding数据。但是用的cudnn的接口是处理unpadding数据的接口,虽然可以调用原有API进行计算,但是计算的结果和精度都是存在问题的。目前自有模型库仅存在1个调用原有API进行计算,且为多层的计算,应该修改为使用新接口计算。

  • 外部用户因为API计算错误不可能在用。附上两个外部用户提出的issue #24300 #22979

  • 所以计划lstm op在2.0会进行大幅度的修改。以后也不会推荐用老的API接口,而是使用新增的API接口。

指派人
分配到
审核者
Request review from
无
里程碑
无
分配里程碑
工时统计
标识: paddlepaddle/Paddle!27193
Source branch: github/fork/GaoWei8/seq_length_input
渝ICP备2023009037号

京公网安备11010502055752号

网络110报警服务 Powered by GitLab CE v13.7
开源知识
Git 入门 Pro Git 电子书 在线学 Git
Markdown 基础入门 IT 技术知识开源图谱
帮助
使用手册 反馈建议 博客
《GitCode 隐私声明》 《GitCode 服务条款》 关于GitCode
Powered by GitLab CE v13.7