Skip to content

Video Privacy Mode#294

Merged
cmpajot merged 6 commits intomasterfrom
claire/video-privacy-mode
May 6, 2022
Merged

Video Privacy Mode#294
cmpajot merged 6 commits intomasterfrom
claire/video-privacy-mode

Conversation

@cmpajot
Copy link
Contributor

@cmpajot cmpajot commented May 4, 2022

Adds privacy mode support for video and removes attachment_type arg. Added integration tests that can be tested with associated scaleAPI PR using poetry run pytest tests/test_scene.py.

Uploaded a dataset on prodclone using privacy mode here: /nucleus/ds_c9szwyv6jb8r3extvxj0?spoof=claire.pajot%40scale.com.

Will update readme with changes to arguments when this is deployed.

[sc-383721]

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #383721: Privacy mode for Video.

Claire Pajot added 2 commits May 4, 2022 16:06
@cmpajot cmpajot marked this pull request as ready for review May 5, 2022 16:55
@cmpajot cmpajot requested a review from tetranoir May 5, 2022 17:00
Copy link
Contributor

@tetranoir tetranoir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cmpajot cmpajot merged commit e99dee2 into master May 6, 2022
@cmpajot cmpajot deleted the claire/video-privacy-mode branch May 6, 2022 22:44
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.

2 participants