[GitHub] [incubator-mxnet] access2rohit commented on pull request #17882: Improve performance of broadcast_axis on CPU
access2rohit commented on pull request #17882: URL: https://github.com/apache/incubator-mxnet/pull/17882#issuecomment-648596357 @apeforest I am investigating the windows CI timeout on my PR. In the meantime can you review ? 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18478: [v1.7.x]License checker enhancement
mxnet-bot commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648590678 Jenkins CI successfully triggered : [edge, windows-cpu, windows-gpu, unix-gpu, unix-cpu, centos-gpu, miscellaneous, sanity, clang, website, centos-cpu] 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
[GitHub] [incubator-mxnet] ciyongch commented on pull request #18478: [v1.7.x]License checker enhancement
ciyongch commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648590594 @mxnet-bot run ci [all] 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18478: [v1.7.x]License checker enhancement
mxnet-bot commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648566251 Jenkins CI successfully triggered : [sanity] 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
[GitHub] [incubator-mxnet] ciyongch commented on pull request #18478: [v1.7.x]License checker enhancement
ciyongch commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648566201 @mxnet-bot run ci [sanity] 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
[GitHub] [incubator-mxnet] ys2843 commented on pull request #18587: prevent master version for installation widget
ys2843 commented on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648563172 > > > 1.6 --> Linux > > > Scala, Java, Clojure, R... and the rest don't display anything for instructions. > > > > > > @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. > > It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 > > @ys2843 on the public version https://mxnet.apache.org/get_started?platform=linux=r=cpu=build-from-source;, it displays "You will need to R v3.4.4+ and build MXNet from source. Please follow the instructions linked above." etc. But in your PR review, the sentence is removed. This is a bug, but not from this PR. Public website also won't show this message by clicking through the widget, it only shows by visiting the url directly. Same as preview. For example, start from https://mxnet.apache.org/get_started? and click through the installation widget (v1.6 => R => GPU), this message won't show. And visit this link directly http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/get_started?platform=linux=r=cpu=build-from-source;. The message shows. I am not sure how this bug was imported, will fix it in another PR. 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
[GitHub] [incubator-mxnet] e0lithic commented on issue #16379: Fail to run example code with Raspberry Pi
e0lithic commented on issue #16379: URL: https://github.com/apache/incubator-mxnet/issues/16379#issuecomment-648561608 @aGiant or @pedrooct Could you kindly share the reference link for the complete set of instructions for building mxnet on raspberry pi. Thanks 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18614: [numpy] FFI flip, rollaxis, stack
mxnet-bot commented on pull request #18614: URL: https://github.com/apache/incubator-mxnet/pull/18614#issuecomment-648557638 Hey @Yiyan66 , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: - To trigger all jobs: @mxnet-bot run ci [all] - To trigger specific jobs: @mxnet-bot run ci [job1, job2] *** **CI supported jobs**: [unix-cpu, windows-cpu, miscellaneous, sanity, centos-gpu, unix-gpu, clang, website, centos-cpu, windows-gpu, edge] *** _Note_: Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. All CI tests must pass before the PR can be merged. 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
[GitHub] [incubator-mxnet] Yiyan66 opened a new pull request #18614: [numpy] FFI flip, rollaxis, stack
Yiyan66 opened a new pull request #18614: URL: https://github.com/apache/incubator-mxnet/pull/18614 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) created (except PRs with tiny changes) - [ ] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage: - Unit tests are added for small changes to verify correctness (e.g. adding a new operator) - Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore) - Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL) - [ ] Code is well-documented: - For user-facing API changes, API doc string has been updated. - For new C++ functions in header files, their functionalities and arguments are documented. - For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable - Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html - [ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change ### Changes ### - [ ] Feature1, tests, (and when applicable, API doc) - [ ] Feature2, tests, (and when applicable, API doc) ## Comments ## - If this change is a backward incompatible change, why must this change be made. - Interesting edge cases to note here 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
[GitHub] [incubator-mxnet] leezu edited a comment on pull request #18587: prevent master version for installation widget
leezu edited a comment on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648544783 > > 1.6 --> Linux > > Scala, Java, Clojure, R... and the rest don't display anything for instructions. > > @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. > It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 @ys2843 on the public version https://mxnet.apache.org/get_started?platform=linux=r=cpu=build-from-source;, it displays "You will need to R v3.4.4+ and build MXNet from source. Please follow the instructions linked above." etc. But in your PR review, the sentence is removed. 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
[incubator-mxnet] branch master updated (4b86c32 -> acf2d27)
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4b86c32 Allow input reordering duing Gluon / CachedOp graph transformations (#17949) add acf2d27 Update tutorials (#18609) No new revisions were added by this update. Summary of changes: docs/python_docs/python/api/gluon/index.rst| 7 +- .../python/tutorials/extend/custom_layer.md| 28 ++--- .../python/tutorials/extend/customop.md| 2 +- .../getting-started/crash-course/6-use_gpus.md | 4 +- .../gluon_from_experiment_to_deployment.md | 3 +- .../logistic_regression_explained.md | 9 +- .../tutorials/getting-started/to-mxnet/pytorch.md | 9 +- .../packages/gluon/blocks/custom-layer.md | 4 +- .../gluon/blocks/custom_layer_beginners.md | 28 ++--- .../tutorials/packages/gluon/blocks/naming.md | 130 ++--- .../python/tutorials/packages/gluon/blocks/nn.md | 3 +- .../packages/gluon/blocks/save_load_params.md | 33 +++--- .../tutorials/packages/gluon/data/datasets.md | 7 +- .../tutorials/packages/gluon/image/info_gan.md | 82 +++-- .../python/tutorials/packages/gluon/image/mnist.md | 22 ++-- .../packages/gluon/image/pretrained_models.md | 3 +- .../tutorials/packages/gluon/loss/custom-loss.md | 16 ++- .../tutorials/packages/gluon/text/transformer.rst | 4 +- .../learning_rates/learning_rate_schedules.md | 31 +++-- .../packages/ndarray/sparse/train_gluon.md | 30 +++-- .../tutorials/packages/onnx/fine_tuning_gluon.md | 5 +- .../python/tutorials/performance/backend/amp.md| 4 +- .../tutorials/performance/backend/profiler.md | 17 ++- 23 files changed, 194 insertions(+), 287 deletions(-)
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18612: [WIP]B axis improv cpu
mxnet-bot commented on pull request #18612: URL: https://github.com/apache/incubator-mxnet/pull/18612#issuecomment-648545314 Jenkins CI successfully triggered : [windows-cpu] 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
[GitHub] [incubator-mxnet] access2rohit commented on pull request #18612: [WIP]B axis improv cpu
access2rohit commented on pull request #18612: URL: https://github.com/apache/incubator-mxnet/pull/18612#issuecomment-648545283 @mxnet-bot run ci [windows-cpu] 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
[incubator-mxnet] branch master updated (4b86c32 -> acf2d27)
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4b86c32 Allow input reordering duing Gluon / CachedOp graph transformations (#17949) add acf2d27 Update tutorials (#18609) No new revisions were added by this update. Summary of changes: docs/python_docs/python/api/gluon/index.rst| 7 +- .../python/tutorials/extend/custom_layer.md| 28 ++--- .../python/tutorials/extend/customop.md| 2 +- .../getting-started/crash-course/6-use_gpus.md | 4 +- .../gluon_from_experiment_to_deployment.md | 3 +- .../logistic_regression_explained.md | 9 +- .../tutorials/getting-started/to-mxnet/pytorch.md | 9 +- .../packages/gluon/blocks/custom-layer.md | 4 +- .../gluon/blocks/custom_layer_beginners.md | 28 ++--- .../tutorials/packages/gluon/blocks/naming.md | 130 ++--- .../python/tutorials/packages/gluon/blocks/nn.md | 3 +- .../packages/gluon/blocks/save_load_params.md | 33 +++--- .../tutorials/packages/gluon/data/datasets.md | 7 +- .../tutorials/packages/gluon/image/info_gan.md | 82 +++-- .../python/tutorials/packages/gluon/image/mnist.md | 22 ++-- .../packages/gluon/image/pretrained_models.md | 3 +- .../tutorials/packages/gluon/loss/custom-loss.md | 16 ++- .../tutorials/packages/gluon/text/transformer.rst | 4 +- .../learning_rates/learning_rate_schedules.md | 31 +++-- .../packages/ndarray/sparse/train_gluon.md | 30 +++-- .../tutorials/packages/onnx/fine_tuning_gluon.md | 5 +- .../python/tutorials/performance/backend/amp.md| 4 +- .../tutorials/performance/backend/profiler.md | 17 ++- 23 files changed, 194 insertions(+), 287 deletions(-)
[GitHub] [incubator-mxnet] leezu commented on pull request #18587: prevent master version for installation widget
leezu commented on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648544783 > > 1.6 --> Linux > > Scala, Java, Clojure, R... and the rest don't display anything for instructions. > > @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. > It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 @ys2843 on the master version, it displays "You will need to R v3.4.4+ and build MXNet from source. Please follow the instructions linked above." etc. But in your PR review, the sentence is removed. 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
[GitHub] [incubator-mxnet] leezu merged pull request #18609: Update docs according to new Block APIs
leezu merged pull request #18609: URL: https://github.com/apache/incubator-mxnet/pull/18609 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
[GitHub] [incubator-mxnet] ciyongch commented on pull request #18585: [v1.7] Cherry pick Increase staggered build timeout (#18568)
ciyongch commented on pull request #18585: URL: https://github.com/apache/incubator-mxnet/pull/18585#issuecomment-648537331 Thanks for the update @josephevans . Ping @leezu @marcoabreu @aaronmarkham to have a review and help merge. Thanks! 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
[GitHub] [incubator-mxnet] ciyongch commented on pull request #18478: [v1.7.x]License checker enhancement
ciyongch commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648535738 Hi @ChaiBapchya , I've updated the numpy version constraint to `< 1.19.0` instead of a fixed version for ci scripts as well as `tests/requirements.txt`, seems it's used by the [windows-cpu] job. Let's wait for the results. 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
[GitHub] [incubator-mxnet] szhengac commented on pull request #18532: add epsilon to adamax
szhengac commented on pull request #18532: URL: https://github.com/apache/incubator-mxnet/pull/18532#issuecomment-648515817 @mxnet-bot run ci [unix-cpu] 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18532: add epsilon to adamax
mxnet-bot commented on pull request #18532: URL: https://github.com/apache/incubator-mxnet/pull/18532#issuecomment-648515867 Jenkins CI successfully triggered : [unix-cpu] 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
[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 5c70b02 Bump the publish timestamp. 5c70b02 is described below commit 5c70b0215d3adf6a49f72746382a723f39cfa403 Author: mxnet-ci AuthorDate: Wed Jun 24 00:40:38 2020 + Bump the publish timestamp. --- date.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/date.txt b/date.txt new file mode 100644 index 000..82be439 --- /dev/null +++ b/date.txt @@ -0,0 +1 @@ +Wed Jun 24 00:40:38 UTC 2020
[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new e972685 Publish triggered by CI e972685 is described below commit e9726854bfff07993cdb3708c01960afa2102964 Author: mxnet-ci AuthorDate: Wed Jun 24 00:40:28 2020 + Publish triggered by CI --- date.txt | 1 - feed.xml | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/date.txt b/date.txt deleted file mode 100644 index 4e32447db..000 --- a/date.txt +++ /dev/null @@ -1 +0,0 @@ -Tue Jun 23 18:41:01 UTC 2020 diff --git a/feed.xml b/feed.xml index c60951c..d263aee 100644 --- a/feed.xml +++ b/feed.xml @@ -1 +1 @@ -http://www.w3.org/2005/Atom; >https://jekyllrb.com/; version="4.0.0">Jekyllhttps://mxnet.apache.org/feed.xml; rel="self" type="application/atom+xml" />https://mxnet.apache.org/; rel="alternate" type="text/html" />2020-06-23T18:30:35+00:00https://mxnet.apache.org/feed.xmlApache MXNetA flexible and efficient library for deep [...] \ No newline at end of file +http://www.w3.org/2005/Atom; >https://jekyllrb.com/; version="4.0.0">Jekyllhttps://mxnet.apache.org/feed.xml; rel="self" type="application/atom+xml" />https://mxnet.apache.org/; rel="alternate" type="text/html" />2020-06-24T00:30:10+00:00https://mxnet.apache.org/feed.xmlApache MXNetA flexible and efficient library for deep [...] \ No newline at end of file
[GitHub] [incubator-mxnet] szhengac commented on a change in pull request #18403: Gluon.probability
szhengac commented on a change in pull request #18403: URL: https://github.com/apache/incubator-mxnet/pull/18403#discussion_r444581289 ## File path: python/mxnet/gluon/probability/transformation/transformation.py ## @@ -0,0 +1,289 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +# coding: utf-8 +"""Transformation Classes""" +__all__ = ["Transformation", "TransformBlock","ComposeTransform", "ExpTransform", + "AffineTransform", "PowerTransform", "AbsTransform", 'SigmoidTransform', + 'SoftmaxTransform'] + +from ..distributions.utils import _clip_prob, cached_property, sum_right_most +from ...block import HybridBlock +import weakref + + +class Transformation(object): +r"""Abstract class for implementing invertible transformation +with computable log det jacobians + +Attributes +-- +bijective : bool + +""" +bijective = False +event_dim = 0 + +def __init__(self, F=None): +self._inv = None +self._F = F +super(Transformation, self).__init__() + +@property +def F(self): +return self._F + +@F.setter +def F(self, value): +self._F = value + +@property +def sign(self): +""" +Returns the sign of the determinant of the Jacobian. +""" +raise NotImplementedError + +@property +def inv(self): +inv = None +if self._inv is not None: +inv = self._inv() +if inv is None: +inv = _InverseTransformation(self) +self._inv = weakref.ref(inv) Review comment: yes 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
[GitHub] [incubator-mxnet] ys2843 commented on pull request #18587: add version check on installation guide
ys2843 commented on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648486956 > > > 1.6 --> Linux > > > Scala, Java, Clojure, R... and the rest don't display anything for instructions. > > > > > > @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. > > It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 > > As far as I'm concerned, that installation widget is now broken. If the purpose of the PR is prevent unsupported versions from showing up then it shouldn't have a customer click down a path of options to have no result. > > If the purpose is to "prevent master" then I guess you could update the title and restate the purpose of the PR... then we deal with how poor the CX is now for customers in another issue/pr. This PR is to prevent any unsupported version (or random string) being entered from URL, which is a bug for the widget. I will restate the purpose and we can discuss what's the current expectation for installation widget with mxnet team. 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
[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18605: Clipboard refactor
ys2843 edited a comment on pull request #18605: URL: https://github.com/apache/incubator-mxnet/pull/18605#issuecomment-648374298 > http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/get_started?platform=linux=python=gpu=pip; @aaronmarkham I tested it with all tutorials and installation guide. No prompt copied and the indents of the code block are preserved For example, http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/api/scala.html 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
[GitHub] [incubator-mxnet] aaronmarkham commented on pull request #18587: add version check on installation guide
aaronmarkham commented on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648484040 > > 1.6 --> Linux > > Scala, Java, Clojure, R... and the rest don't display anything for instructions. > > @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. > It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 As far as I'm concerned, that installation widget is now broken. If the purpose of the PR is prevent unsupported versions from showing up then it shouldn't have a customer click down a path of options to have no result. If the purpose is to "prevent master" then I guess you could update the title and restate the purpose of the PR... then we deal with how poor the CX is now for customers in another issue/pr. 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
[GitHub] [incubator-mxnet] harrysekim edited a comment on issue #18611: Failed to build from source with "OSError: exception: access violation writing 0x0000000000000000"
harrysekim edited a comment on issue #18611: URL: https://github.com/apache/incubator-mxnet/issues/18611#issuecomment-648477452 @szha I have tried several things and copy all dependents dll(include not dependents dll) file to both folder(build/Release, cpp-package/scripts) i check remove each dll file from floder and try build. Finally I got the list of required dll files. In my case, I built it from the beginning in the order above and it works fine. 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
[GitHub] [incubator-mxnet] harrysekim commented on issue #18611: Failed to build from source with "OSError: exception: access violation writing 0x0000000000000000"
harrysekim commented on issue #18611: URL: https://github.com/apache/incubator-mxnet/issues/18611#issuecomment-648477452 @szha I have tried several things and copy all dependents dll(include not dependents dll) file to both folder(build/Release, cpp-package/scripts) i check remove each dll file from floder and try build. Finally I got the list of required dll files. 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
[GitHub] [incubator-mxnet] szha commented on issue #18611: Failed to build from source with "OSError: exception: access violation writing 0x0000000000000000"
szha commented on issue #18611: URL: https://github.com/apache/incubator-mxnet/issues/18611#issuecomment-648471028 @harrysekim did you figure out what happened? 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
[GitHub] [incubator-mxnet] harrysekim closed issue #18611: Failed to build from source with "OSError: exception: access violation writing 0x0000000000000000"
harrysekim closed issue #18611: URL: https://github.com/apache/incubator-mxnet/issues/18611 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
[GitHub] [incubator-mxnet] ys2843 commented on pull request #18613: Fix Julia API reference and tutorials redirect on master
ys2843 commented on pull request #18613: URL: https://github.com/apache/incubator-mxnet/pull/18613#issuecomment-648449273 @mxnet-label-bot add [Website] 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18613: Fix Julia API reference and tutorials redirect on master
mxnet-bot commented on pull request #18613: URL: https://github.com/apache/incubator-mxnet/pull/18613#issuecomment-648431640 Hey @ys2843 , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: - To trigger all jobs: @mxnet-bot run ci [all] - To trigger specific jobs: @mxnet-bot run ci [job1, job2] *** **CI supported jobs**: [centos-gpu, sanity, miscellaneous, unix-cpu, edge, website, windows-cpu, centos-cpu, unix-gpu, windows-gpu, clang] *** _Note_: Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. All CI tests must pass before the PR can be merged. 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
[GitHub] [incubator-mxnet] ys2843 opened a new pull request #18613: Fix Julia API reference and tutorials redirect on master
ys2843 opened a new pull request #18613: URL: https://github.com/apache/incubator-mxnet/pull/18613 ## Description ## The redirect rules in #18607 for Julia does not work as expected. ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) created (except PRs with tiny changes) - [ ] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage: ### Changes ### - [ ] Fix Julia redirect rule: master => v1.6 ## Comments ## - Preview: http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/api 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
[GitHub] [incubator-mxnet] sxjscience commented on a change in pull request #18603: DeepNumpy HuberLoss
sxjscience commented on a change in pull request #18603: URL: https://github.com/apache/incubator-mxnet/pull/18603#discussion_r444507043 ## File path: python/mxnet/gluon/loss.py ## @@ -650,9 +650,22 @@ def __init__(self, margin=1, weight=None, batch_axis=0, **kwargs): def hybrid_forward(self, F, pred, label, sample_weight=None): label = _reshape_like(F, label, pred) -loss = F.relu(self._margin - pred * label) -loss = _apply_weighting(F, loss, self._weight, sample_weight) -return F.mean(loss, axis=self._batch_axis, exclude=True) + +if is_np_array(): +loss = F.np.abs(label - pred) +loss = F.np.where(loss > self._rho, loss - 0.5 * self._rho, +(0.5 / self._rho) * F.np.square(loss)) +loss = _apply_weighting(F, loss, self._weight, sample_weight) +if F is ndarray: +return F.np.mean(loss, axis=tuple(range(1, loss.ndim))) +else: +return F.npx.batch_flatten(loss).mean(axis=1) Review comment: This won't work for arbitrary batch axis. 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
[incubator-mxnet] branch master updated (74fcb99 -> 4b86c32)
This is an automated email from the ASF dual-hosted git repository. haibin pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 74fcb99 redirect api reference on v-master to v1.6 (#18607) add 4b86c32 Allow input reordering duing Gluon / CachedOp graph transformations (#17949) No new revisions were added by this update. Summary of changes: src/imperative/cached_op.cc| 58 ++ src/imperative/cached_op.h | 34 +--- src/imperative/cached_op_threadsafe.cc | 4 ++- src/imperative/naive_cached_op.cc | 3 +- tests/python/gpu/test_fusion.py| 35 5 files changed, 94 insertions(+), 40 deletions(-)
[incubator-mxnet] branch master updated: Allow input reordering duing Gluon / CachedOp graph transformations (#17949)
This is an automated email from the ASF dual-hosted git repository. haibin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 4b86c32 Allow input reordering duing Gluon / CachedOp graph transformations (#17949) 4b86c32 is described below commit 4b86c32832a994e76b97dfc58c8a672db87e721d Author: mk-61 <56651474+mk...@users.noreply.github.com> AuthorDate: Tue Jun 23 13:49:06 2020 -0700 Allow input reordering duing Gluon / CachedOp graph transformations (#17949) * Initial commit of input reordering in Gluon * Add test for Gluon input reorder * Fix backward in CachedOp for input reordering * Fix test_input_reorder for backward pass * Fix merge error in NaiveCachedOp * Include correct header for std::iota Co-authored-by: Vladimir Cherepanov --- src/imperative/cached_op.cc| 58 ++ src/imperative/cached_op.h | 34 +--- src/imperative/cached_op_threadsafe.cc | 4 ++- src/imperative/naive_cached_op.cc | 3 +- tests/python/gpu/test_fusion.py| 35 5 files changed, 94 insertions(+), 40 deletions(-) diff --git a/src/imperative/cached_op.cc b/src/imperative/cached_op.cc index 83e8d31..7b3a5d3 100644 --- a/src/imperative/cached_op.cc +++ b/src/imperative/cached_op.cc @@ -147,10 +147,9 @@ bool CachedOp::CheckDynamicShapeExists(const Context& default_ctx, auto& state = state_ptr.get_state(); nnvm::Graph& g = state.info.fwd_graph; - ShapeVector shape_inputs; - shape_inputs.reserve(inputs.size()); - for (auto input : inputs) { -shape_inputs.emplace_back(input->shape()); + ShapeVector shape_inputs(inputs.size()); + for (size_t i = 0; i < inputs.size(); ++i) { +shape_inputs[i] = inputs[state.info.input_map[i]]->shape(); } // We leverage the shape inference pass to detect whether dynamic shape exists. // If so, the pass will fail with `contain_dynamic_shape = true`, @@ -176,16 +175,13 @@ bool CachedOp::SetForwardGraph( CHECK_EQ(inputs.size(), num_inputs()); nnvm::Graph& g = info->fwd_graph; - ShapeVector shape_inputs; - DTypeVector dtype_inputs; - StorageTypeVector storage_type_inputs; - shape_inputs.reserve(inputs.size()); - dtype_inputs.reserve(inputs.size()); - storage_type_inputs.reserve(inputs.size()); - for (auto input : inputs) { -shape_inputs.emplace_back(input->shape()); -dtype_inputs.emplace_back(input->dtype()); -storage_type_inputs.emplace_back(input->storage_type()); + ShapeVector shape_inputs(inputs.size()); + DTypeVector dtype_inputs(inputs.size()); + StorageTypeVector storage_type_inputs(inputs.size()); + for (size_t i = 0; i < inputs.size(); ++i) { +shape_inputs[i] = inputs[info->input_map[i]]->shape(); +dtype_inputs[i] = inputs[info->input_map[i]]->dtype(); +storage_type_inputs[i] = inputs[info->input_map[i]]->storage_type(); } bool match = true; @@ -321,9 +317,10 @@ bool CachedOp::SetBackwardGraph( if (info->bwd_input_eid[i] == kEidNotExist) { continue; } -shapes[info->bwd_input_eid[i]] = inputs[i]->shape(); -dtypes[info->bwd_input_eid[i]] = inputs[i]->dtype(); -stypes[info->bwd_input_eid[i]] = inputs[i]->storage_type(); +size_t oi = BwdOriginalInput(info->input_map, i); +shapes[info->bwd_input_eid[i]] = inputs[oi]->shape(); +dtypes[info->bwd_input_eid[i]] = inputs[oi]->dtype(); +stypes[info->bwd_input_eid[i]] = inputs[oi]->storage_type(); } std::pair node_range, entry_range; @@ -649,22 +646,22 @@ OpStatePtr CachedOp::StaticForward( if (config_.static_shape) { for (auto i : config_.param_indices) { auto nid = idx.input_nodes()[i]; - if (!arrays[idx.entry_id(nid, 0)]->IsSame(*inputs[i])) { + if (!arrays[idx.entry_id(nid, 0)]->IsSame(*inputs[state.info.input_map[i]])) { match = false; auto ptr = [idx.entry_id(nid, 0)]; CHECK_EQ(arrays[idx.entry_id(nid, 0)], ptr); -*arrays[idx.entry_id(nid, 0)] = *inputs[i]; +*arrays[idx.entry_id(nid, 0)] = *inputs[state.info.input_map[i]]; state.dynamic_entries[idx.entry_id(nid, 0)] = false; } } for (auto i : config_.data_indices) { auto eid = idx.entry_id(idx.input_nodes()[i], 0); - arrays[eid] = inputs[i]; + arrays[eid] = inputs[state.info.input_map[i]]; } } else { for (size_t i = 0; i < num_inputs(); ++i) { auto nid = idx.input_nodes()[i]; - arrays[idx.entry_id(nid, 0)] = inputs[i]; + arrays[idx.entry_id(nid, 0)] = inputs[state.info.input_map[i]]; } } @@ -714,6 +711,7 @@ OpStatePtr CachedOp::DynamicForward( std::lock_guard lock(state.mutex); SetForwardGraph(default_ctx, , recording, inputs); runtime.info.fwd_graph = state.info.fwd_graph; +runtime.info.input_map =
[GitHub] [incubator-mxnet] eric-haibin-lin merged pull request #17949: Allow input reordering duing Gluon / CachedOp graph transformations
eric-haibin-lin merged pull request #17949: URL: https://github.com/apache/incubator-mxnet/pull/17949 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18612: [WIP]B axis improv cpu
mxnet-bot commented on pull request #18612: URL: https://github.com/apache/incubator-mxnet/pull/18612#issuecomment-648396096 Hey @access2rohit , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: - To trigger all jobs: @mxnet-bot run ci [all] - To trigger specific jobs: @mxnet-bot run ci [job1, job2] *** **CI supported jobs**: [unix-gpu, windows-gpu, unix-cpu, clang, edge, sanity, centos-gpu, centos-cpu, windows-cpu, website, miscellaneous] *** _Note_: Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. All CI tests must pass before the PR can be merged. 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
[GitHub] [incubator-mxnet] access2rohit opened a new pull request #18612: [WIP]B axis improv cpu
access2rohit opened a new pull request #18612: URL: https://github.com/apache/incubator-mxnet/pull/18612 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) created (except PRs with tiny changes) - [ ] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage: - Unit tests are added for small changes to verify correctness (e.g. adding a new operator) - Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore) - Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL) - [ ] Code is well-documented: - For user-facing API changes, API doc string has been updated. - For new C++ functions in header files, their functionalities and arguments are documented. - For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable - Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html - [ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change ### Changes ### - [ ] Feature1, tests, (and when applicable, API doc) - [ ] Feature2, tests, (and when applicable, API doc) ## Comments ## - If this change is a backward incompatible change, why must this change be made. - Interesting edge cases to note here 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
[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18587: add version check on installation guide
ys2843 edited a comment on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648373376 > 1.6 --> Linux > Scala, Java, Clojure, R... and the rest don't display anything for instructions. @aaronmarkham This PR doesn't touch the docs, these instructions are removed in #18487 . In Production, these docs are also inaccessible. It checks the version with the version select dropdown, only redirects to 1.6 when the string in url is not valid. So as long as “master" is in the dropdown, it won’t redirects to v1.6 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
[GitHub] [incubator-mxnet] ys2843 commented on pull request #18605: Clipboard refactor
ys2843 commented on pull request #18605: URL: https://github.com/apache/incubator-mxnet/pull/18605#issuecomment-648374298 > http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/get_started?platform=linux=python=gpu=pip; @aaronmarkham I tested it with all tutorials and installation guide. No prompt copied and the indent of the code are preserved 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
[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18605: Clipboard refactor
ys2843 edited a comment on pull request #18605: URL: https://github.com/apache/incubator-mxnet/pull/18605#issuecomment-648374298 > http://ec2-34-219-134-42.us-west-2.compute.amazonaws.com/get_started?platform=linux=python=gpu=pip; @aaronmarkham I tested it with all tutorials and installation guide. No prompt copied and the indents of the code block are preserved 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
[GitHub] [incubator-mxnet] ys2843 commented on pull request #18587: add version check on installation guide
ys2843 commented on pull request #18587: URL: https://github.com/apache/incubator-mxnet/pull/18587#issuecomment-648373376 > 1.6 --> Linux > Scala, Java, Clojure, R... and the rest don't display anything for instructions. This PR doesn't touch the docs, these instructions are removed in #18487 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18586: [v1.6] Cherry pick Increase staggered build timeout (#18568)
mxnet-bot commented on pull request #18586: URL: https://github.com/apache/incubator-mxnet/pull/18586#issuecomment-648347585 Jenkins CI successfully triggered : [edge, unix-gpu] 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
[GitHub] [incubator-mxnet] josephevans commented on pull request #18585: [v1.7] Cherry pick Increase staggered build timeout (#18568)
josephevans commented on pull request #18585: URL: https://github.com/apache/incubator-mxnet/pull/18585#issuecomment-648347268 > Hi @josephevans , please help to re-trigger the `unix-gpu` job as it's stuck. > The modification increases the full time from 30min to 60 min, while decrease the sanity time from 180min to 60min, seems not align to the description? Hi @ciyongch, we actually increased the overall "full" job which first triggers sanity build, then if it passes we run the rest of the jobs. But since it doesn't make sense to have a longer timeout for sanity build than the "full" build, we made them the same. If the sanity build timed out, the full build would have already timed out of the timeout is longer. 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
[GitHub] [incubator-mxnet] josephevans commented on pull request #18586: [v1.6] Cherry pick Increase staggered build timeout (#18568)
josephevans commented on pull request #18586: URL: https://github.com/apache/incubator-mxnet/pull/18586#issuecomment-648347539 @mxnet-bot run ci [edge, unix-gpu] 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18585: [v1.7] Cherry pick Increase staggered build timeout (#18568)
mxnet-bot commented on pull request #18585: URL: https://github.com/apache/incubator-mxnet/pull/18585#issuecomment-648346187 Jenkins CI successfully triggered : [unix-gpu] 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
[GitHub] [incubator-mxnet] josephevans commented on pull request #18585: [v1.7] Cherry pick Increase staggered build timeout (#18568)
josephevans commented on pull request #18585: URL: https://github.com/apache/incubator-mxnet/pull/18585#issuecomment-648346117 @mxnet-bot run ci [unix-gpu] 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
[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new e24ecf3 Bump the publish timestamp. e24ecf3 is described below commit e24ecf319e633d3551a82eedcf95e286df60c9ad Author: mxnet-ci AuthorDate: Tue Jun 23 18:41:01 2020 + Bump the publish timestamp. --- date.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/date.txt b/date.txt new file mode 100644 index 000..4e32447db --- /dev/null +++ b/date.txt @@ -0,0 +1 @@ +Tue Jun 23 18:41:01 UTC 2020
[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new a88b726 Publish triggered by CI a88b726 is described below commit a88b72691fefbf53a4816a06134bff7d6eac23f6 Author: mxnet-ci AuthorDate: Tue Jun 23 18:40:51 2020 + Publish triggered by CI --- date.txt | 1 - feed.xml | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/date.txt b/date.txt deleted file mode 100644 index 7d2ad0f..000 --- a/date.txt +++ /dev/null @@ -1 +0,0 @@ -Tue Jun 23 12:40:32 UTC 2020 diff --git a/feed.xml b/feed.xml index 123d3ff..c60951c 100644 --- a/feed.xml +++ b/feed.xml @@ -1 +1 @@ -http://www.w3.org/2005/Atom; >https://jekyllrb.com/; version="4.0.0">Jekyllhttps://mxnet.apache.org/feed.xml; rel="self" type="application/atom+xml" />https://mxnet.apache.org/; rel="alternate" type="text/html" />2020-06-23T12:30:08+00:00https://mxnet.apache.org/feed.xmlApache MXNetA flexible and efficient library for deep [...] \ No newline at end of file +http://www.w3.org/2005/Atom; >https://jekyllrb.com/; version="4.0.0">Jekyllhttps://mxnet.apache.org/feed.xml; rel="self" type="application/atom+xml" />https://mxnet.apache.org/; rel="alternate" type="text/html" />2020-06-23T18:30:35+00:00https://mxnet.apache.org/feed.xmlApache MXNetA flexible and efficient library for deep [...] \ No newline at end of file
[GitHub] [incubator-mxnet] RuRo commented on pull request #18601: Use new mxnet.gluon.blocks APIs for ONNX gluon tests
RuRo commented on pull request #18601: URL: https://github.com/apache/incubator-mxnet/pull/18601#issuecomment-648322589 @szha I am not sure, what do you mean? Can you elaborate? It seems to me, that the errors I am seeing in CI are just the "regular" flaky mxnet tests/killed containers. 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
[GitHub] [incubator-mxnet] RuRo removed a comment on pull request #18601: Use new mxnet.gluon.blocks APIs for ONNX gluon tests
RuRo removed a comment on pull request #18601: URL: https://github.com/apache/incubator-mxnet/pull/18601#issuecomment-648322723 @mxnet-bot run ci [centos-cpu] 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
[GitHub] [incubator-mxnet] RuRo commented on pull request #18601: Use new mxnet.gluon.blocks APIs for ONNX gluon tests
RuRo commented on pull request #18601: URL: https://github.com/apache/incubator-mxnet/pull/18601#issuecomment-648322723 @mxnet-bot run ci [centos-cpu] 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18601: Use new mxnet.gluon.blocks APIs for ONNX gluon tests
mxnet-bot commented on pull request #18601: URL: https://github.com/apache/incubator-mxnet/pull/18601#issuecomment-648322769 Jenkins CI successfully triggered : [centos-cpu] 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
[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18478: [v1.7.x]License checker enhancement
ChaiBapchya commented on pull request #18478: URL: https://github.com/apache/incubator-mxnet/pull/18478#issuecomment-648312575 Thanks for driving this. Looks like fixing numpy version helped resolve other pipeline issues [barring windows-cpu] @leezu any thoughts? 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
[GitHub] [incubator-mxnet] stu1130 commented on pull request #18602: Fix softmax, logsoftmax failed on empty ndarray
stu1130 commented on pull request #18602: URL: https://github.com/apache/incubator-mxnet/pull/18602#issuecomment-648291838 @bgawrych Yes, so you don't need to take care of this edge case on the mkldnn integration part 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
[GitHub] [incubator-mxnet] ys2843 commented on a change in pull request #18598: graph executor c api removal
ys2843 commented on a change in pull request #18598: URL: https://github.com/apache/incubator-mxnet/pull/18598#discussion_r444362937 ## File path: ci/jenkins/Jenkinsfile_website_full ## @@ -40,13 +40,7 @@ core_logic: { utils.parallel_stage('Build Docs', [ custom_steps.docs_jekyll(), -custom_steps.docs_c('libmxnet'), custom_steps.docs_python('libmxnet'), -custom_steps.docs_julia('libmxnet'), Review comment: Other versions of website except master are archived static files and their build process does not depend on any branch. 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
[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new cde1608 Publish triggered by CI cde1608 is described below commit cde1608e82e48960fe6bad2cbc0c6c1a0b8e79c3 Author: mxnet-ci AuthorDate: Tue Jun 23 12:40:20 2020 + Publish triggered by CI --- .../docs/_modules/mxnet/gluon/parameter.html | 60 +++--- date.txt | 1 - feed.xml | 2 +- 3 files changed, 31 insertions(+), 32 deletions(-) diff --git a/api/python/docs/_modules/mxnet/gluon/parameter.html b/api/python/docs/_modules/mxnet/gluon/parameter.html index 8bcdda1..6889939 100644 --- a/api/python/docs/_modules/mxnet/gluon/parameter.html +++ b/api/python/docs/_modules/mxnet/gluon/parameter.html @@ -759,7 +759,7 @@ Error for unfinished deferred initialization. pass -class Parameter(object): +[docs]class Parameter(object): A Container holding parameters (weights) of Blocks. :py:class:`Parameter` holds a copy of the parameter on each :py:class:`Context` after @@ -1129,7 +1129,7 @@ trainer._row_sparse_pull(self, data, all_row_ids, full_idx=True) return data -def initialize(self, init=None, ctx=None, default_init=initializer.Uniform[docs] def initialize(self, init=None, ctx=Noneforce_reinit=False, structural_name=): Initializes parameter and gradient arrays. Only used for :py:class:`NDArray` API. @@ -1198,9 +1198,9 @@ invalid shape: %s.%(self.name, str(self.shape))) self._deferred_init = (init, ctx, default_init, None) -self._finish_deferred_init() +self._finish_deferred_init() -def reset_ctx(self, ctx): +[docs] def reset_ctx(self, ctx): Re-assign Parameter to other contexts. Parameters @@ -1222,9 +1222,9 @@ self._deferred_init = (init, ctx, default_init, data) else: raise ValueError(Cannot reset context for Parameter %s because it - has not been initialized.%self.name) + has not been initialized.%self.name) -def set_data(self, data): +[docs] def set_data(self, data): Sets this parameters value on all contexts. self.shape = data.shape @@ -1241,9 +1241,9 @@ trainer._reset_kvstore() for arr in self._check_and_get(self._data, list): -arr[:] = data +arr[:] = data -def row_sparse_data(self, row_id): +[docs] def row_sparse_data(self, row_id): Returns a copy of the row_sparse parameter on the same context as row_ids. The copy only retains rows whose ids occur in provided row ids. The parameter must have been initialized on this context before. @@ -1261,9 +1261,9 @@ raise RuntimeError(Cannot return a copy of Parameter %s via row_sparse_data() \ because its storage type is %s. Please use data() instead. \ %(self.name, self._stype)) -return self._get_row_sparse(self._data, row_id.ctx, row_id) +return self._get_row_sparse(self._data, row_id.ctx, row_id) -def list_row_sparse_data(self, row_id): +[docs] def list_row_sparse_data(self, row_id): Returns copies of the row_sparse parameter on all contexts, in the same order as creation. The copy only retains rows whose ids occur in provided row ids. The parameter must have been initialized before. @@ -1281,9 +1281,9 @@ raise RuntimeError(Cannot return copies of Parameter %s on all contexts via \ list_row_sparse_data() because its storage type is %s. Please \ use data() instead. % (self.name, self._stype)) -return self._get_row_sparse(self._data, list, row_id) +return self._get_row_sparse(self._data, list, row_id) -def data(self, ctx=None): +[docs] def data(self, ctx=None): Returns a copy of this parameter on one context. Must have been initialized on this context before. For sparse parameters, use :py:meth:`Parameter.row_sparse_data` instead. @@ -1303,9 +1303,9 @@ instead. % (self.name, str(ctx), self._stype)) data = self._check_and_get(self._data, ctx) dc.set_variable(data, self.var()) -return data +return data -def list_data(self): +[docs] def list_data(self): Returns copies of this parameter on all contexts, in the same order as creation. For sparse parameters, use :py:meth:`Parameter.list_row_sparse_data` instead. @@ -1318,9 +1318,9 @@
[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new b5bcabd Bump the publish timestamp. b5bcabd is described below commit b5bcabd1cb0ecd64bd394faf153de50f58e1ff50 Author: mxnet-ci AuthorDate: Tue Jun 23 12:40:32 2020 + Bump the publish timestamp. --- date.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/date.txt b/date.txt new file mode 100644 index 000..7d2ad0f --- /dev/null +++ b/date.txt @@ -0,0 +1 @@ +Tue Jun 23 12:40:32 UTC 2020
[GitHub] [incubator-mxnet] harrysekim opened a new issue #18611: Failed to build from source with "OSError: exception: access violation writing 0x0000000000000000"
harrysekim opened a new issue #18611: URL: https://github.com/apache/incubator-mxnet/issues/18611 ## Description I 'm trying to compile mxnet from source code in windows10, vs2017 1. mxnet clone tag 1.6.0 git clone --recursive -b 1.6.0 https://github.com/apache/incubator-mxnet mxnet_1.6.0 2. set opencv, openblas env set OpenCV_DIR=C:\opencv-3.4.1_with_freetype2 set OpenBLAS_HOME=C:\OpenBLAS-0.3.7-x64 3. cmake cmake -G "Visual Studio 15 2017 Win64" -T cuda=10.1,host=x64 -DUSE_CUDA=1 -DBUILD_CPP_EXAMPLES=1 -DUSE_CPP_PACKAGE=1 -DUSE_CUDNN=1 -DUSE_NVRTC=1 -DUSE_OPENCV=1 -DUSE_OPENMP=1 -DUSE_BLAS=open -DUSE_LAPACK=1 -DUSE_DIST_KVSTORE=0 -DCUDA_ARCH_LIST=Common -DCUDA_TOOLSET=10.1 -DCUDNN_INCLUDE="C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.1\include" -DCUDNN_LIBRARY="C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.1\lib\x64\cudnn.lib" "../" 4. build "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\MSBuild.exe" mxnet.sln /p:Configuration=Release;Platform=x64 /maxcpucount ### Error Message Traceback (most recent call last): File "OpWrapperGenerator.py", line 433, in raise(e) File "OpWrapperGenerator.py", line 427, in f.write(patternStr % ParseAllOps()) File "OpWrapperGenerator.py", line 321, in ParseAllOps cdll.libmxnet = cdll.LoadLibrary(sys.argv[1]) File "C:\Users\admin\AppData\Local\Programs\Python\Python38\lib\ctypes\__init__.py", line 451, in LoadLibrary return self._dlltype(name) File "C:\Users\admin\AppData\Local\Programs\Python\Python38\lib\ctypes\__init__.py", line 373, in __init__ self._handle = _dlopen(self._name, mode) FileNotFoundError: Could not find module 'C:\Workspace\mxnet_1.6.0\build\Release\libmxnet.dll' (or one of its dependencies). Try using the full path with constructor syntax. 5. copy dependents dll file to Release folder cublas64_10.dll cudart64_101.dll cudnn64_7.dll cufft64_10.dll curand64_10.dll cusolver64_10.dll nvrtc64_101_0.dll libgcc_s_seh-1.dll libgfortran-3.dll libopenblas.dll libquadmath-0.dll nvcuda.dll opencv_world341.dll 6. rebuild ### Error Message "Running: OpWrapperGenerator.py" Traceback (most recent call last): File "OpWrapperGenerator.py", line 433, in raise(e) File "OpWrapperGenerator.py", line 427, in f.write(patternStr % ParseAllOps()) File "OpWrapperGenerator.py", line 338, in ParseAllOps r = ListOP(byref(nOps), byref(opHandlers)) OSError: exception: access violation writing 0x ## Environment ``` python 3.8.3 compiler : vs2017 # paste outputs here ``` "Running: OpWrapperGenerator.py" Traceback (most recent call last): File "OpWrapperGenerator.py", line 433, in raise(e) File "OpWrapperGenerator.py", line 427, in f.write(patternStr % ParseAllOps()) File "OpWrapperGenerator.py", line 338, in ParseAllOps r = ListOP(byref(nOps), byref(opHandlers)) OSError: exception: access violation writing 0x 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
[GitHub] [incubator-mxnet] RoyiAvital commented on issue #8382: MATLAB API of Mxnet
RoyiAvital commented on issue #8382: URL: https://github.com/apache/incubator-mxnet/issues/8382#issuecomment-648009740 The README file says there is MATLAB compatibility yet I can't find any documentation on how to use it? Do you still support MATLAB? 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
[GitHub] [incubator-mxnet] bgawrych commented on pull request #18602: Fix softmax, logsoftmax failed on empty ndarray
bgawrych commented on pull request #18602: URL: https://github.com/apache/incubator-mxnet/pull/18602#issuecomment-648002694 > We usually just check and return directly before we launch the kernel which is run against different contexts > > ``` > if (inputs[0].shape().Size() == 0U) return; > ``` > > As the bug also affect on GPU and all the other contexts if possible So this condition should be put in the beginning of FCompute and FComputeEx functions, right? @stu1130 @TaoLv 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
[GitHub] [incubator-mxnet] hzfan commented on pull request #18546: [Numpy] FFI: tril_indices
hzfan commented on pull request #18546: URL: https://github.com/apache/incubator-mxnet/pull/18546#issuecomment-647969823 CI's green. @yzhliu 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
[GitHub] [incubator-mxnet] anko-intel commented on issue #14357: [Bug] Batchnorm running_var behaves differently when using gpu vs. cpu
anko-intel commented on issue #14357: URL: https://github.com/apache/incubator-mxnet/issues/14357#issuecomment-647960540 @adrianloy, @PatricZhao, could you close the issue as the results on master (959143696) are the same on GPU and CPU (see the comments above for details)? 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
[GitHub] [incubator-mxnet] anko-intel opened a new issue #18610: BatchNorm running variables are not properly synchronized on CPU
anko-intel opened a new issue #18610: URL: https://github.com/apache/incubator-mxnet/issues/18610 ## Description During investigation of #14357 I have found that running_mean and running_var variables are sometimes not yet calculated when converted to NumPy. ### Error Message When runing the python test from https://github.com/apache/incubator-mxnet/issues/14357#issuecomment-644887485 sometimes there is information that consecutive reads of the same variable differes, which means that backward pass is calculated after the first read: `var1 = layer2.running_var.data().asnumpy()` It shouldn't happen as asnumpy() function should force all needed calculations to get running_var values. I such case the test shows a message similar to the one below: ``` seed: 331752807 cpu(0) - DIFF in running_var reads: 0 : [3.9974644 3.9977627 3.997775 ] 2 : [3.9962945 4.0025434 3.9987571] cpu(0) [3.9962945 4.0025434 3.9987571] [10.001289 9.997483 10.002162] gpu(0) [3.9962947 4.0025434 3.9987576] [10.001292 9.997481 10.002162] ``` Verified on master branch 959143696 ## To Reproduce run python test from https://github.com/apache/incubator-mxnet/issues/14357#issuecomment-644887485 ## What have you tried to solve it? I already have the fix. I will put it to github soon. 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
[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18609: Update docs according to new Block APIs
mxnet-bot commented on pull request #18609: URL: https://github.com/apache/incubator-mxnet/pull/18609#issuecomment-647942368 Hey @acphile , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands: - To trigger all jobs: @mxnet-bot run ci [all] - To trigger specific jobs: @mxnet-bot run ci [job1, job2] *** **CI supported jobs**: [unix-cpu, centos-gpu, centos-cpu, website, sanity, miscellaneous, edge, windows-gpu, unix-gpu, clang, windows-cpu] *** _Note_: Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin. All CI tests must pass before the PR can be merged. 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
[GitHub] [incubator-mxnet] acphile opened a new pull request #18609: Update docs according to new Block APIs
acphile opened a new pull request #18609: URL: https://github.com/apache/incubator-mxnet/pull/18609 ## Description ## Update docs according to new Block APIs (https://github.com/apache/incubator-mxnet/pull/18413) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) created (except PRs with tiny changes) - [ ] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage: - Unit tests are added for small changes to verify correctness (e.g. adding a new operator) - Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore) - Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL) - [ ] Code is well-documented: - For user-facing API changes, API doc string has been updated. - For new C++ functions in header files, their functionalities and arguments are documented. - For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable - Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html - [ ] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change ### Changes ### - [ ] Feature1, tests, (and when applicable, API doc) - [ ] Feature2, tests, (and when applicable, API doc) ## Comments ## - If this change is a backward incompatible change, why must this change be made. - Interesting edge cases to note here 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
[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 8fa68f6 Publish triggered by CI 8fa68f6 is described below commit 8fa68f6768508b3e880547be617cf16a3732ff3e Author: mxnet-ci AuthorDate: Tue Jun 23 06:41:25 2020 + Publish triggered by CI --- .htaccess | 8 +++ .../docs/_modules/mxnet/gluon/parameter.html | 60 +++--- date.txt | 1 - feed.xml | 2 +- 4 files changed, 39 insertions(+), 32 deletions(-) diff --git a/.htaccess b/.htaccess index ba40eff..b0d4d49 100644 --- a/.htaccess +++ b/.htaccess @@ -22,6 +22,14 @@ RewriteOptions AllowNoSlash +# TODO temporary fix for issue #18604 +Redirect 302 /api/r/docs/api/R-package/build/mxnet-r-reference-manual.pdf https://mxnet-public.s3.us-east-2.amazonaws.com/docs/v1.x/mxnet-r-reference-manual.pdf +Redirect 302 /api/scala/docs/api/ /versions/1.6/api/scala/docs/api/ +Redirect 302 /api/java/docs/api/ /versions/1.6/api/java/docs/api/ +Redirect 302 /api/clojure/docs/api/ /versions/1.6/api/clojure/docs/api/ +Redirect 302 /api/cpp/docs/api/ /versions/1.6/api/cpp/docs/api/ +Redirect 302 /api/julia/docs/api/ /versions/1.6/api/julia/docs/api/ + # Redirect Chinese visitors to Chinese CDN, temporary solution for slow site speed in China RewriteCond %{ENV:GEOIP_COUNTRY_CODE} ^CN$ RewriteCond %{HTTP_HOST} !cdn diff --git a/api/python/docs/_modules/mxnet/gluon/parameter.html b/api/python/docs/_modules/mxnet/gluon/parameter.html index 6889939..8bcdda1 100644 --- a/api/python/docs/_modules/mxnet/gluon/parameter.html +++ b/api/python/docs/_modules/mxnet/gluon/parameter.html @@ -759,7 +759,7 @@ Error for unfinished deferred initialization. pass -[docs]class Parameter(object): +class Parameter(object): A Container holding parameters (weights) of Blocks. :py:class:`Parameter` holds a copy of the parameter on each :py:class:`Context` after @@ -1129,7 +1129,7 @@ trainer._row_sparse_pull(self, data, all_row_ids, full_idx=True) return data -[docs] def initialize(self, init=None, ctx=Nonedef initialize(self, init=None, ctx=None, default_init=initializer.Uniformforce_reinit=False, structural_name=): Initializes parameter and gradient arrays. Only used for :py:class:`NDArray` API. @@ -1198,9 +1198,9 @@ invalid shape: %s.%(self.name, str(self.shape))) self._deferred_init = (init, ctx, default_init, None) -self._finish_deferred_init() +self._finish_deferred_init() -[docs] def reset_ctx(self, ctx): +def reset_ctx(self, ctx): Re-assign Parameter to other contexts. Parameters @@ -1222,9 +1222,9 @@ self._deferred_init = (init, ctx, default_init, data) else: raise ValueError(Cannot reset context for Parameter %s because it - has not been initialized.%self.name) + has not been initialized.%self.name) -[docs] def set_data(self, data): +def set_data(self, data): Sets this parameters value on all contexts. self.shape = data.shape @@ -1241,9 +1241,9 @@ trainer._reset_kvstore() for arr in self._check_and_get(self._data, list): -arr[:] = data +arr[:] = data -[docs] def row_sparse_data(self, row_id): +def row_sparse_data(self, row_id): Returns a copy of the row_sparse parameter on the same context as row_ids. The copy only retains rows whose ids occur in provided row ids. The parameter must have been initialized on this context before. @@ -1261,9 +1261,9 @@ raise RuntimeError(Cannot return a copy of Parameter %s via row_sparse_data() \ because its storage type is %s. Please use data() instead. \ %(self.name, self._stype)) -return self._get_row_sparse(self._data, row_id.ctx, row_id) +return self._get_row_sparse(self._data, row_id.ctx, row_id) -[docs] def list_row_sparse_data(self, row_id): +def list_row_sparse_data(self, row_id): Returns copies of the row_sparse parameter on all contexts, in the same order as creation. The copy only retains rows whose ids occur in provided row ids. The parameter must have been initialized before. @@ -1281,9 +1281,9 @@ raise RuntimeError(Cannot return copies of Parameter %s on all contexts via \ list_row_sparse_data() because its storage type is %s. Please \ use data() instead. % (self.name,
[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new a853816 Bump the publish timestamp. a853816 is described below commit a8538168047bb57bff31bcd6cb3a97998209c43b Author: mxnet-ci AuthorDate: Tue Jun 23 06:41:30 2020 + Bump the publish timestamp. --- date.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/date.txt b/date.txt new file mode 100644 index 000..f200def --- /dev/null +++ b/date.txt @@ -0,0 +1 @@ +Tue Jun 23 06:41:30 UTC 2020