Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scene “Rough Assembly” preview & Video Player #3

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

audionerd
Copy link
Member

@audionerd audionerd commented Jul 2, 2020

A Rough Assembly preview is a sequence of the “best” Takes for each Shot in a Scene.

The “best” Take is the highest-rated, most recent take for a Shot.

If a Take hasn’t been downloaded yet, it is skipped when preparing the list of "best" Takes.

If no "best" Takes are present, the Video Player is hidden, and a “Not Available” notice is shown.

Users can view the Rough Assembly preview for a Scene from the Scene Detail page.

Tasks

  • Rough Assembly data
  • Video Player
  • Show time progress bar
  • Re-test RTSP downloader on real hardware

Future Work

  • Rough Assembly preview data could be described in a JSON response (e.g.: preview.json) instead of inline HTML data for use by other tools.
  • Store duration of take stream file in database (currently calls ffprobe to calculate for each rough assembly take on every visit to scene detail page)

audionerd added 14 commits July 2, 2020 10:44
- add video player controller js
- add xstate
- refactor "best take" query
- show elapsed time of total playlist duration
- use data-action for event handling (ended, timeupdate)
- don't use CSS transition to animate
- update periodically via JavaScript
- can set period via VideoPlayer.UPDATE_INTERVAL_MS
- determine duration of downloaded RTSP file once complete
- patch metadata_json
- takes.metadata_json can't be NULL
- initialize takes.metadata_json object when take is created
- add rtsp-client test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant