Activate Travis build and fix scripts to comply with flake8 standards

[sbesson]

This PR adds a .travis.yml file at the top-level of the repository running flake8 on all the Python scripts. It also fixes individual scripts to comply with flake8 standards and let the Travis build pass.

• omero/analysis_scripts/Kymograph.py
• omero/analysis_scripts/Kymograph_Analysis.py
• omero/analysis_scripts/Plot_Profile.py
• omero/export_scripts/Batch_Image_Export.py
• omero/export_scripts/Make_Movie.py
• omero/figure_scripts/Movie_Figure.py
• omero/figure_scripts/Movie_ROI_Figure.py
• omero/figure_scripts/ROI_Split_Figure.py
• omero/figure_scripts/Split_View_Figure.py
• omero/figure_scripts/Thumbnail_Figure.py
• omero/import_scripts/Populate_ROI.py
• omero/setup_scripts/FLIM_initialise.py
• omero/util_scripts/Channel_Offsets.py
• omero/util_scripts/Combine_Images.py
• omero/util_scripts/Dataset_To_Plate.py
• omero/util_scripts/Images_From_ROIs.py

To test this PR:

• check the changes do not break any script functionality in the deployed server
• check https://travis-ci.org/sbesson/scripts is green

[joshmoore]

Looking at sbesson@b759912?w=1 is fairly readable. (Note the ?w=1 on the end of that link)

[joshmoore]

@will-moore, once this PR is accepted, I'll turn on travis for all of ome/scripts and then we'll have testing on every new PR.

@sbesson : don't know if this is important but travis log includes:

Could not locate requirements.txt. Override the install: key in your .travis.yml to install dependencies.

[sbesson]

@joshmoore: good point. Fixed in the last 2 commits.

[will-moore]

Running Movie_ROI_Figure.py I get

Traceback (most recent call last):
  File "./script", line 843, in <module>
    runAsScript()
  File "./script", line 832, in runAsScript
    fileAnnotation, message = roiFigure(conn, commandArgs)
  File "./script", line 564, in roiFigure
    imageLabels.append(getLabels(name, tagsList, pdList))
UnboundLocalError: local variable 'getLabels' referenced before assignment

[will-moore]

Same for Movie_Figure.py

Traceback (most recent call last):
  File "./script", line 672, in <module>
    runAsScript()
  File "./script", line 662, in runAsScript
    fileAnnotation, message = movieFigure(conn, commandArgs)
  File "./script", line 430, in movieFigure
    imageLabels.append(getLabels(name, tagsList, pdList))
UnboundLocalError: local variable 'getLabels' referenced before assignment

[will-moore]

Having run almost all of the scripts at least once, the only errors I get are the two getLabels ones above, but I also found a problem with unwrapping parameters on Combine_Images.py, which I think is unrelated: Ticket created: https://trac.openmicroscopy.org.uk/ome/ticket/11473

[will-moore]

OK - working now. Good to merge.

[joshmoore]

@sbesson: did you want to wait for any rebasing due to the develop PR you mentioned?

[sbesson]

Up to @will-moore , the PR in question is #31 which needs/could be rebased on dev_4_4.
IN all cases, it only concerns one section of one file so fixing conflicts should be straightforward.

[will-moore]

I have rebased #31 to #45. Merge in whichever order makes things easiest. I'm happy to fix any conflicts if you want to get this one in first?

[joshmoore]

@will-moore: 👍 thanks. I'm merging this. Could you rebase and rationalize the rebase with 45? I'll go ahead and activate the travis build for ome/scripts so that we'll see the test status in your rebased PR.

[sbesson]

--rebased-to #46