Support for Video Attributes

[benjaminpkane]

Not blocking anything, but it would be nice to have soon.

[lethosor]

I can look into it if you have labels with video-level attrs laying around. Shouldn't be too complicated to add.

[brimoor]

@lethosor this touches on the notion of "rendering VideoLabels at the frame-level" that I mentioned previously.

The way to support this would be to implement the render() method here: https://github.com/voxel51/eta/blob/0096e7983d01e87d6d88bb8f8cf193fbb5cfbbf1/eta/core/video.py#L1396

Let's chat when time permits.

[lethosor]

Player51 isn't structured the same way as ETA (i.e. there's no real equivalent to the render method you're referring to), but I was thinking of essentially treating video attributes as frame-level attributes present on every frame. There are a couple places where I think that could be done cleanly/easily.

[brimoor]

I know that Player51 isn't structured the same way. I'm basically saying that, once you see what new VideoLabels can support, you will likely want a render method.

Video attributes are simple enough to hack by copying on every frame, yes. That's what the render() method I pointed to does, as well.

[benjaminpkane]

#94 has a new attrs.html test with video, frame, and object attributes of all three types.