[GitHub] [incubator-singa] dcslin commented on issue #514: SINGA-482 tc comprehension integration
dcslin commented on issue #514: SINGA-482 tc comprehension integration URL: https://github.com/apache/incubator-singa/pull/514#issuecomment-534047644 Integration is done except support CPU. CPU support development is done, however the behavior was not expected even in clean TC env. CPU backed TC return zero tensor as shown in https://gist.githubusercontent.com/dcslin/f3ed411012c144163f3bedbe621257f6/raw/9f6af3f5e62b5bb52a58c087967c813b3acd8872/tensor-comprehension-cpu-errors This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-singa] dcslin commented on issue #514: SINGA-482 tc comprehension integration
dcslin commented on issue #514: SINGA-482 tc comprehension integration URL: https://github.com/apache/incubator-singa/pull/514#issuecomment-523808529 TODO: 1. move tc execution code into device `DoExec`, and support CPU 2. package dependencies as conda package 3. integer support This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-singa] dcslin commented on issue #514: SINGA-482 tc comprehension integration
dcslin commented on issue #514: SINGA-482 tc comprehension integration URL: https://github.com/apache/incubator-singa/pull/514#issuecomment-521572874 Hi, this is the working version of tc integration for now, with following notes: 1. tested in the environment specified in the dockerfile `cuda9` 2. it implemented `MatMul`, `Relu` in TC for demonstration and usage is shown in following tests: ` ./bin/test_singa --gtest_filter=*TC*`, and tests are passing. 3. for now only working in Cuda environment P.S. `softmax` is implemented but still checking why the output is not correct. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services