Use LGTM and flake8 to find some missing imports

[cclauss]

• Should ScratchPad() instead be ScratchPadCommandLines()?
• __save_movie()__ can be found in upstream.

flake8 testing of https://github.com/IBM/mi-prometheus on Python 3.7.1

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./miprometheus/problems/seq_to_seq/algorithmic/recall/scratch_pad_cl.py:176:18: F821 undefined name 'ScratchPad'
    scratchpad = ScratchPad(params)
                 ^
./miprometheus/problems/seq_to_seq/vqa/cog/cog_utils/stim_generator.py:758:5: F821 undefined name 'save_movie'
    save_movie(movie, save_name, t_total)
    ^
./miprometheus/problems/seq_to_seq/vqa/cog/cog_utils/stim_generator.py:797:5: F821 undefined name 'save_movie'
    save_movie(movie, save_name, t_total)
    ^
3     F821 undefined name 'ScratchPad'
3

E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc. Most other flake8 issues are merely "style violations" -- useful for readability but they do not effect runtime safety.

• F821: undefined name name
• F822: undefined name name in __all__
• F823: local variable name referenced before assignment
• E901: SyntaxError or IndentationError
• E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree

[tkornuta-ibm]

Hi, @cclauss thanks for your work, we were participating in NeurIPS last week, will investigate the raised issue as soon as possible.