Jasmine
Results
1
comments of
Jasmine
问题:ExpressionLayer在实现Forward()接口时,使用了抽象表达式->词法解析->语法解析&生成表达式->后序遍历生成后缀表达式->计算后缀表达式的值,个人觉得比较冗余。 个人思考:作者能否考虑使用:抽象表达式->词法解析获取前缀表达式->计算前缀表达式的值? 理由: 1. 从数据结构上来讲这样是可行的。 2. 这样减少了一些操作可以节省一定的空间和时间,而且Resnet&Yolo等模型都有较多的ExpressionLayer,从模型总体执行时间考虑能起到一定的速度提升和性能提升。 说明:本人已修改代码,测试了在Resnet和Yolo模型上能输出与原来相同的结果,目前已经提交Pr,期望被采纳。