From dev-return-5553-archive-asf-public=cust-asf.ponee.io@singa.apache.org Mon Jun 1 14:16:20 2020 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id B39031804BB for ; Mon, 1 Jun 2020 16:16:19 +0200 (CEST) Received: (qmail 54276 invoked by uid 500); 1 Jun 2020 14:16:19 -0000 Mailing-List: contact dev-help@singa.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@singa.apache.org Delivered-To: mailing list dev@singa.apache.org Received: (qmail 54266 invoked by uid 99); 1 Jun 2020 14:16:19 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 01 Jun 2020 14:16:19 +0000 From: =?utf-8?q?GitBox?= To: dev@singa.apache.org Subject: =?utf-8?q?=5BGitHub=5D_=5Bsinga=5D_XJDKC_edited_a_comment_on_pull_request_?= =?utf-8?q?=23697=3A_New_Model_Layer_Operator_API?= Message-ID: <159102097896.10999.1874559134748474747.asfpy@gitbox.apache.org> Date: Mon, 01 Jun 2020 14:16:18 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit In-Reply-To: References: XJDKC edited a comment on pull request #697: URL: https://github.com/apache/singa/pull/697#issuecomment-636885818 > As discussed, last commit changes: 1 Restore creator settings for interim tensors; 2 update device_check() after new get states implementation; 3 fix typo in LSTM; 4 fix test case for save model and load model, as new impl does not keep top level layer name; 5 remove proof of concept script. > Retested test_model.py, test_operations.py. > From my side, it looks ok to go except memory issue to be fix. Thanks, @dcslin. * For 4, It's because the DoubleLinear in the test case does not meet specifications. Change \_\_call\_\_ to forward will solve this problem, I will update it. * I have checked the models in examples, now they can be trained correctly. * Since the merge resulted in many duplicate commits, I will clean it up by rebasing the branch. * I will remove converter.py, lsgan.py, vanilla.py * I will fix examples/mlp/module.py Hi, @joddiy. pls help to fix these linting errors. https://travis-ci.org/github/apache/singa/jobs/693422214#L2361 ---------------------------------------------------------------- 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