Seg3D icon indicating copy to clipboard operation
Seg3D copied to clipboard

Layer export needs refactoring

Open a-y-khan opened this issue 10 years ago • 3 comments

  • [ ] Unit tests for exporter base class
  • [ ] Unit tests for export actions
  • [ ] Consolidate cut and paste exporter code for managing label values in exporter child classes
  • [ ] Consolidate cut and paste code for filling in background mask layer data block in exporter child classes
  • [ ] Refactor ActionExportSegmentation so action code is in run method, and not validate
  • [ ] ActionExportSegmentation layers argument as vector of strings makes more sense
  • [ ] Add optional ActionExportSegmentation argument values that contains list of label values
  • [ ] Exporter type not set in ActionExportSegmentation::Dispatch methods
  • [ ] Check possibly unused string processing code in SegmentationExportWizard

a-y-khan avatar Jul 07 '15 16:07 a-y-khan

Consider splitting up ActionExportSegmentation - the code path that takes the fully configured file exporter does not need the same arguments and the path that takes the list of layers.

a-y-khan avatar Jul 07 '15 19:07 a-y-khan

See also #70.

a-y-khan avatar Jul 07 '15 23:07 a-y-khan

#287

allywarner avatar Jul 17 '18 20:07 allywarner