pi-camera-connect icon indicating copy to clipboard operation
pi-camera-connect copied to clipboard

Added some missing options

Open GastonZalba opened this issue 2 years ago • 3 comments

Shared options

  • videoStabilization
  • colorEffect
  • dynamicRange
  • imageEffectMode
  • statistics
  • meteringMode
  • flickerMode
  • roi
  • awbGains
  • annotate
  • annotateExtra

Still options

  • raw
  • quality
  • thumbnail
  • burst
  • exif
  • gpsExif

Added HQ Camera mode

GastonZalba avatar Jan 25 '23 20:01 GastonZalba

Hi, excelent. I'm going to look at this and come back.

Another thing that would seem appropriate to do (i can make a separate PR) would be to move all the enums to a separate file and then reexport them from index.ts. So, the current behaviour would be the same, but it will allow to import only the enums (using something like pi-camera-connect/dist/@enums) from a frontend appplication (like react/electron) without requiring the entire library and preventing the fatal import errors from child_process and stream dependencies.

Sorry if my english is a bit broken.

GastonZalba avatar Feb 22 '23 01:02 GastonZalba

@gregnr changes applied. Let me know what you think about the previous enums suggestion (move them to a separate file). I'm also working on enabling the preview window and arguments (fullscreen, preview, keypress and dispnum). I'll be back in a few days with that.

GastonZalba avatar Feb 22 '23 23:02 GastonZalba

@gregnr Hi, i finally added a few more options (annotate, exif and related), expanded a little the still test, and cleaned up some mess that i left after the last requested changes. In the README I also modified some values that I had set as "default" from null to undefined, which I understand is more correct (and added some missing info as well). Let me know if any further adjustment is needed

GastonZalba avatar Feb 26 '23 22:02 GastonZalba