Needs More Testing: Export Layer context menu item

[ilastik]

Needs More Testing: Export Layer context menu item

[stuarteberg]

Eliminate "unecessary controls" (e.g. roi selection). Test all remaining controls in the window.... ideally in a unit test.

[akreshuk]

it takes a very long time for the initial widget to come up and the gui is not responsive at that time. This does not fell right.

[akreshuk]

When trying to export a 2d image to png, it just hangs.

[thorbenk]

The reason for "it just hangs" was wrong lazyflow usage, where the entire data was requested before instantiating the dialog.

[ukoethe]

Buote re-wrote the dialog, but the tests are still missing. => Assigned to Buote.

[akreshuk]

Would be good to have better command line progress info or remove it altogether. And replace the ETA abbreviation by something meaningful.

[stuarteberg]

The most recent iteration of the export window needs unit tests. A quick example of how to begin writing such a test can be found in the if __name__ == "__main__": section of dataExportOptionsDlg.py.

The test should verify:

• That the dialog configures the operator with the expected settings.
• That the dialog does not crash

The test does not need to verify:

• That the export actually works (this is done in a test of the export operator in lazyflow)

[stuarteberg]

@kamay did a lot of testing here, and the underlying operators also have some unit tests. Looks good enough to me.