Skip to content

Conversation

@anna-charlotte
Copy link
Contributor

@anna-charlotte anna-charlotte commented Feb 15, 2023

Add display for image, audio, video:

For all the following, add options for when displayed in notebook. Also make possible for remote urls as well as local files.

  • ImageTensor

  • ImageUrl

  • AudioTensor

  • AudioUrl

  • VideoTensor

  • VideoUrl

  • check and update documentation, if required. See guide

@anna-charlotte anna-charlotte changed the title Feat display img audio vid feat: display image audio and video from url and tensor Feb 15, 2023
@anna-charlotte anna-charlotte mentioned this pull request Feb 15, 2023
47 tasks
anna-charlotte added 23 commits February 15, 2023 16:18
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
anna-charlotte added 5 commits February 15, 2023 16:50
@anna-charlotte anna-charlotte force-pushed the feat-display-img-audio-vid branch from 00e1130 to 442dd29 Compare February 16, 2023 09:22
@github-actions github-actions bot added size/m and removed size/l labels Feb 16, 2023
anna-charlotte added 3 commits February 16, 2023 15:26
Signed-off-by: anna-charlotte <[email protected]>
Signed-off-by: anna-charlotte <[email protected]>
@anna-charlotte anna-charlotte marked this pull request as ready for review February 16, 2023 15:35
:return: bytes
"""
bytes = BytesIO()
self.save(
Copy link
Contributor Author

@anna-charlotte anna-charlotte Feb 16, 2023

Choose a reason for hiding this comment

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

Here mypy complains because "T" has no attribute "save". Why does it no complain though in line 164 (b = self.to_bytes(audio_tensor=audio)), isn't to_bytes() just a method of VideoTensorMixin like save()?

Signed-off-by: anna-charlotte <[email protected]>
@github-actions
Copy link

📝 Docs are deployed on https://ft-feat-display-img-audio-vid--jina-docs.netlify.app 🎉

@samsja samsja merged commit a90850d into feat-rewrite-v2 Feb 17, 2023
@samsja samsja deleted the feat-display-img-audio-vid branch February 17, 2023 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants