You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@singa.apache.org by GitBox <gi...@apache.org> on 2020/07/24 06:54:11 UTC

[GitHub] [singa] dcslin opened a new pull request #772: qabot update

dcslin opened a new pull request #772:
URL: https://github.com/apache/singa/pull/772


   updated qabot training and data scripts
   added max, mean, mlp qabot models,
   updated ranking loss fn in autograd
   fix bug in cudnn rnn in autograd
   added some utils tensor fn(random, zeros, ones)
   added cudnn rnn set param api
   fixed and added test to autograd mse loss cos sim, reduce mean


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] dcslin commented on pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
dcslin commented on pull request #772:
URL: https://github.com/apache/singa/pull/772#issuecomment-663375950


   Previous bug was not discovered due to loose constraints set in testings, now the testing check values upto 7th significant digit.
   Now the result is ok and working as expected: https://gist.github.com/dcslin/85be2aed0a78478c6675115c191b315a
   Example scripts are:
   1. `python3 examples/qabot/qabot_data.py` download data
   2.  `python3 examples/qabot/qabot_train.py` do training and testing
   
   Next step is to add Embedding layer.
   
   This PR is ready for review. @nudles . Thank you.


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] lgtm-com[bot] commented on pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #772:
URL: https://github.com/apache/singa/pull/772#issuecomment-664107985


   This pull request **introduces 10 alerts** and **fixes 5** when merging 89f5b0f6a1f0436ace8ed6c130628a3f7c7e851c into 5f4b250c1ef4d0380769310e3884bf0c1115d2a1 - [view on LGTM.com](https://lgtm.com/projects/g/apache/singa/rev/pr-77e3e19a7a09ea6c10e6801c1b0c8ae646dc46ae)
   
   **new alerts:**
   
   * 5 for Unused import
   * 3 for Variable defined multiple times
   * 2 for &#39;import \*&#39; may pollute namespace
   
   **fixed alerts:**
   
   * 4 for Unused import
   * 1 for Unused local variable


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] lgtm-com[bot] commented on pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #772:
URL: https://github.com/apache/singa/pull/772#issuecomment-664763496






----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] nudles merged pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
nudles merged pull request #772:
URL: https://github.com/apache/singa/pull/772


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] lgtm-com[bot] commented on pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #772:
URL: https://github.com/apache/singa/pull/772#issuecomment-663379784


   This pull request **introduces 10 alerts** and **fixes 5** when merging eda75d3ae03b4a960b3b04388f18d645e98b27e3 into 5f4b250c1ef4d0380769310e3884bf0c1115d2a1 - [view on LGTM.com](https://lgtm.com/projects/g/apache/singa/rev/pr-215d0e4a694cc4496d4edd2abd9ee13dd90298b3)
   
   **new alerts:**
   
   * 5 for Unused import
   * 3 for Variable defined multiple times
   * 2 for &#39;import \*&#39; may pollute namespace
   
   **fixed alerts:**
   
   * 4 for Unused import
   * 1 for Unused local variable


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [singa] dcslin commented on pull request #772: qabot update

Posted by GitBox <gi...@apache.org>.
dcslin commented on pull request #772:
URL: https://github.com/apache/singa/pull/772#issuecomment-664735124






----------------------------------------------------------------
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:
users@infra.apache.org