[RSDK-9573] Autodetect height and width of source cam in video store

README.md

@@ -52,6 +52,9 @@ On the new component panel, copy and paste the following attribute template into
   }
 }
 ```
+
+Additionally, make sure to add your configured data manager service to the `depends_on` array of your `video-store` component.
+
 > For more information, see [Configure a Machine](https://docs.viam.com/manage/configuration/).
 
 ### Attributes
@@ -71,9 +74,9 @@ On the new component panel, copy and paste the following attribute template into
 |                 | `bitrate`         | integer | optional  | Throughput of encoder in bits per second. Higher for better quality video, and lower for better storage efficiency. |
 |                 | `preset`          | string  | optional  | Name of codec video preset to use. See [here](https://trac.ffmpeg.org/wiki/Encode/H.264#a2.Chooseapresetandtune) for preset options.                                                                |
 | `cam_props`     |                   | object  | required  |                                                                                                   |
-|                 | `width`           | integer | required  | Width of the source camera frames in pixels.                                                      |
-|                 | `height`          | integer | required  | Height of the source camera frames in pixels.                                                     |
-|                 | `framerate`       | integer | required  | Number of frames per second provided by the source camera.                                        |
+|                 | `width`           | integer | optional  | Width of the source camera frames in pixels. If unspecified, will try to autodetect by fetching a frame from the source camera.                                                    |
+|                 | `height`          | integer | optional  | Height of the source camera frames in pixels. If unspecified, will try to autodetect by fetching a frame from the source camera.                                                     |
+|                 | `framerate`       | integer | required  | Number of frames per second provided by the source camera. Unlike width and height, this is a required attribute, and cannot be autodetected.                                                                            |
 
 ### Example Configuration
 

cam/cam.go

@@ -36,11 +36,12 @@ const (
 	defaultStoragePath    = ".viam/video-storage"
 	defaultLogLevel       = "error"
 
-	maxGRPCSize     = 1024 * 1024 * 32 // bytes
-	deleterInterval = 10               // minutes
-	retryInterval   = 1                // seconds
-	asyncTimeout    = 60               // seconds
-	tempPath        = "/tmp"
+	maxGRPCSize           = 1024 * 1024 * 32 // bytes
+	deleterInterval       = 10               // minutes
+	retryInterval         = 1                // seconds
+	asyncTimeout          = 60               // seconds
+	numFetchFrameAttempts = 3                // iterations
+	tempPath              = "/tmp"
 )
 
 type videostore struct {
@@ -125,7 +126,7 @@ func init() {
 }
 
 func newvideostore(
-	_ context.Context,
+	ctx context.Context,
 	deps resource.Dependencies,
 	conf resource.Config,
 	logger logging.Logger,
@@ -166,6 +167,27 @@ func newvideostore(
 	if newConf.Video.Format != "" {
 		format = newConf.Video.Format
 	}
+
+	if newConf.Properties.Width == 0 && newConf.Properties.Height == 0 {
+		vs.logger.Info("received unspecified frame width and height, fetching frame to get dimensions")
+		for range make([]struct{}, numFetchFrameAttempts) {
+			frame, err := camera.DecodeImageFromCamera(ctx, rutils.MimeTypeJPEG, nil, vs.cam)
+			if err != nil {
+				vs.logger.Warn("failed to get and decode frame from camera, retrying. Error: ", err)
+				time.Sleep(retryInterval * time.Second)
+				continue
+			}
+			bounds := frame.Bounds()
+			newConf.Properties.Width = bounds.Dx()
+			newConf.Properties.Height = bounds.Dy()
+			vs.logger.Infof("received frame width and height: %d, %d", newConf.Properties.Width, newConf.Properties.Height)
+			break
+		}
+	}
+	if newConf.Properties.Width == 0 && newConf.Properties.Height == 0 {
+		return nil, fmt.Errorf("failed to get source camera width and height after %d attempts", numFetchFrameAttempts)
+	}
+
 	vs.enc, err = newEncoder(
 		logger,
 		codec,