GRIP icon indicating copy to clipboard operation
GRIP copied to clipboard

Improve code generation

Open SamCarlberg opened this issue 8 years ago • 0 comments
trafficstars

UI

  • [x] Remember the previous save location
  • [x] Keep track of the save location (possibly as a project setting)
  • [x] ~~Offer options for "Generate" and "Generate as", similar to the common "Save" and "Save as"~~ pipeline name can be set every time code is generated
  • [x] Offer option to implement WPILib VisionPipeline interface (https://github.com/wpilibsuite/allwpilib/pull/388)
  • Offer custom options on a per-language basis:
    • All
      • [x] Set the pipeline class name
    • Java
      • [x] Package name (currently, there's no package declaration)
    • C++
      • [ ] CUDA option
      • [ ] TAPI option
    • Python
      • [x] Module name (= file name)
      • [x] Pipeline class name (should NOT be the same as the module name)

Generated code

  • [ ] Have a data type containing common properties of contours (center point, width, height, area) and possibly other commonly used OpenCV data types
  • [ ] Make operation inputs from UI controls (threshold values, filters, etc.) be class constants instead of local variables
  • Java (and possible C++)
    • [x] ~~Properly capitalize setsourceX() to setSourceX()~~ setSource mehods were removed
    • [ ] Fix docs
    • [ ] Only import required classes
    • [ ] Make list variables use the List<Foo> type instead of ArrayList<Foo>
      • [ ] Use diamond operator on constructor: List<Foo> foo = new ArrayList<>();
  • C++
    • [x] Put all GRIP stuff in its own namespace
    • [x] Remove all using namespace foo declarations
    • [ ] Only include required headers
  • [ ] Be able to use TAPI
  • [ ] Be able to use CUDA

CUDA API docs: http://docs.opencv.org/3.1.0/d1/d1e/group__cuda.html

SamCarlberg avatar Dec 01 '16 02:12 SamCarlberg