mxnet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hagay Lupesko <lupe...@gmail.com>
Subject Re: Pylint Undefined variable/name error
Date Fri, 27 Jul 2018 04:52:41 GMT
+1 - thanks Vandana!

On Thu, Jul 26, 2018 at 8:05 PM sandeep krishnamurthy <
sandeep.krishna98@gmail.com> wrote:

> Thanks Vandana for picking up this issue. I think this is important to be
> fixed and enabled in CI. Please let us know effort required to fix these
> issues and we all can jump in and help you.
>
> On Thu, Jul 26, 2018, 7:27 PM Vandana Kannan <vandana268@gmail.com> wrote:
>
> > Hi All,
> >
> > On enabling the option "undefined-variable" in pylint (in pylintrc) and
> > executing on the latest code, 52 errors show up (most of them from the
> > example folder). These could lead to Python NameError at runtime. The
> > errors are documented in
> > https://github.com/apache/incubator-mxnet/issues/11904.
> >
> > Currently, this Pylint option is disabled in CI and pylint is not
> executed
> > on the example folder.
> >
> > It might be better to enable this option in CI to catch these errors
> early
> > on, and also work on fixing the errors. Any thoughts/suggestions?
> >
> > Thanks,
> > Vandana
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message