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

WebGL Followup #799

Closed
4 of 5 tasks
tqchen opened this issue Jan 20, 2018 · 7 comments
Closed
4 of 5 tasks

WebGL Followup #799

tqchen opened this issue Jan 20, 2018 · 7 comments

Comments

@tqchen
Copy link
Member

tqchen commented Jan 20, 2018

#672 Provide basic support for WebGL features. This Issue lists the followup items to be done.

  • Store large data into 2D texture
    - We can use canonical form of [1024, y] (always make lowest dimension 1024).
  • GEMM 1024x1024
  • TOPI support of gl functions
  • End to end example of resnet/mobielnet
  • Optimize the bottleneck operators(conv2d_nchw), possibly via intermediate layout change
@tqchen
Copy link
Member Author

tqchen commented Jan 20, 2018

@phisiart

@phisiart
Copy link
Contributor

phisiart commented Jan 27, 2018

  • conv2d_nchw
  • dense
  • injective
  • pool
  • softmax

@tqchen
Copy link
Member Author

tqchen commented Feb 7, 2018

#836 #817

@tqchen
Copy link
Member Author

tqchen commented Feb 18, 2018

dmlc/nnvm#369

@tqchen
Copy link
Member Author

tqchen commented Mar 3, 2018

Now the only item that is left is the perf improvement of conv2d

@phisiart
Copy link
Contributor

phisiart commented Mar 4, 2018

I feel like I can't continue working on this. My full-time job starts on Monday.

I took a look at deeplearn.js today. They don't seem to have special treatment for their WebGL conv2d operator: it's just several layers of for loops. There's no layout transformation either (no transpose or tiling) in model construction.

Last semester when we were doing the course project, we experimented matrix multiplication, and transpose doesn't really change performance. We didn't try tiling though.

@tqchen
Copy link
Member Author

tqchen commented Mar 8, 2018

close this for now as most of stuffs are checked in thanks to @phisiart , followup with perf on another issue

@tqchen tqchen closed this as completed Mar 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants