Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

对课程和项目的建议,请写在这里。 #33

Open
zjhellofss opened this issue Jun 13, 2023 · 4 comments
Open

对课程和项目的建议,请写在这里。 #33

zjhellofss opened this issue Jun 13, 2023 · 4 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@zjhellofss
Copy link
Owner

No description provided.

@zjhellofss zjhellofss added the good first issue Good for newcomers label Jun 13, 2023
@zjhellofss zjhellofss changed the title 对课程和项目的建议 对课程和项目的建议,请写在这里。 Jun 13, 2023
@zjhellofss zjhellofss self-assigned this Jun 14, 2023
@soham24
Copy link

soham24 commented Jul 18, 2023

this is gold-level material, sir.
how can I learn from videos if I am an English speaker /no Chinese

@gogoRcq
Copy link

gogoRcq commented Jul 21, 2023

layer等抽象里面写死了 float,后续怎么对 u_int8进行支持呢

@TypeFloat
Copy link
Contributor

layer等抽象里面写死了 float,后续怎么对 u_int8进行支持呢

借楼向作者提问一下,在实现Tensor时,不同数据类型例如float和uint8,其函数实现都是一样的,为什么不可以在类模版中进行函数的实现,而是在特化类Tensor中进行实现呢?如果有修改的必要,我愿意做这一部分的贡献。

@Jasmine-up
Copy link
Contributor

Jasmine-up commented Sep 14, 2023

问题:ExpressionLayer在实现Forward()接口时,使用了抽象表达式->词法解析->语法解析&生成表达式->后序遍历生成后缀表达式->计算后缀表达式的值,个人觉得比较冗余。
个人思考:作者能否考虑使用:抽象表达式->词法解析获取前缀表达式->计算前缀表达式的值?
理由:

  1. 从数据结构上来讲这样是可行的。
  2. 这样减少了一些操作可以节省一定的空间和时间,而且Resnet&Yolo等模型都有较多的ExpressionLayer,从模型总体执行时间考虑能起到一定的速度提升和性能提升。

说明:本人已修改代码,测试了在Resnet和Yolo模型上能输出与原来相同的结果,目前已经提交Pr,期望被采纳。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

5 participants