SReview
SReview copied to clipboard
Issues with /released metadata
The metadata is attempting to be compatible with debconf-video's archive-meta, but there are some issues:
Conference:
- Would be nice to have
video_base - And
schedule
Formats:
-
video_formatsis a list of dicts, it should be a dict. - The bitrate is an unqualified integer. It should probably be
1024kor1m, not1024. - Missing
container
Talks:
- Speakers is a single string, not a string per speaker.
- Descriptions are
null. I'd prefer to omit it, if you don't know it. -
eventidisn't a defined key.
Here's what I'm doing to fix & improve the data: https://salsa.debian.org/debconf-video-team/archive-meta/blob/master/tools/sreview.py
also, we apparently only list the file as it is produced under the first profile, rather than under all profiles
Also, the "default" format is the "lq" format, and the "webm" format is what we'd call the default.