Skip to content

Support local recording #436

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

Open
19 tasks
patrickelectric opened this issue Sep 2, 2024 · 4 comments · May be fixed by #484
Open
19 tasks

Support local recording #436

patrickelectric opened this issue Sep 2, 2024 · 4 comments · May be fixed by #484

Comments

@patrickelectric
Copy link
Member

Something like the following should be implemented in the camera manager to support local video recording

  • It should support local video recording
    • It should record as a single file or breaking it in 10s
  • Record can be done using MAVLink (MAV_CMD_VIDEO_START_CAPTURE) and via REST API
  • Recording status should be exposed via MAVLink (CAMERA_CAPTURE_STATUS) and via REST API
  • MCM should provide a component id for itself, not the streams, where parameter could be used to control things like mission names
  • Files/Chunks should be store with incremental names \d+.mp4.part (?) inside a "temporary" folder (not true temporary, but for the process) until is complete
  • The folder should be named using an uuid4
  • The folder should move to a finished location once the record is done
  • With the folder, a metadata.json (?) file should be used to store information about the recorded video and session
    • Camera component ID (for order/serialization)
    • Mission name (based on PARAM_EXT_VALUE)
    • Finished status (Done [boolean])
  • MCM should provide a frontend and a rest api where it would be possible to:
    • Start recording
    • List available records
    • Download videos
      • Merge chunks before download
        It would be nicer if:
  • CAMERA_CAPTURE_STATUS includes a stream ID similar to MAV_CMD_VIDEO_START_CAPTURE
  • MAV_CMD_VIDEO_START_CAPTURE includes a human-readable name
  • Component ID (130) is reserved to a camera manager
@abhishek202206
Copy link

@patrickelectric has there any progress made on this request. even if something is loosely written can that be released for beta testing.

@joaoantoniocardoso
Copy link
Collaborator

joaoantoniocardoso commented Apr 30, 2025

@abhishek202206 I just made the current work visible as a draft PR here, but it is not in a functional state yet.

@abhishek202206
Copy link

Thanks @joaoantoniocardoso I can use that in my development.

@joaoantoniocardoso
Copy link
Collaborator

joaoantoniocardoso commented Apr 30, 2025

@abhishek202206 Feel free to open a PR with your findings, thanks!

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 a pull request may close this issue.

3 participants