mxnet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sandeep krishnamurthy <sandeep.krishn...@gmail.com>
Subject Re: Pylint Undefined variable/name error
Date Fri, 27 Jul 2018 03:05:18 GMT
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