The Wayback Machine - https://web.archive.org/web/20220620063749/https://github.com/activeloopai/Hub/issues/1556
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

Jump to bottom

[FEATURE] hub.auto for audio and video htypes #1556

Open
2 tasks
FayazRahman opened this issue Mar 19, 2022 · 7 comments
Open
2 tasks

[FEATURE] hub.auto for audio and video htypes #1556

FayazRahman opened this issue Mar 19, 2022 · 7 comments
Assignees
@neel2299
Labels
enhancement good first issue
Projects
New Contributor projects

Comments

@FayazRahman
Copy link
Collaborator

@FayazRahman FayazRahman commented Mar 19, 2022

🚨🚨 Feature Request

  • Related to an existing Issue
  • A new implementation (Improvement, Extension)

Is your feature request related to a problem?

Extend hub.auto to enable ingesting audio and video data. Take a look at ingesting image datasets for a start.

Description of the possible solution

Adding a video_ingestion.py file to hub/auto/unstructured with necessary changes would work. But this might mean rewriting a lot of code as in image_classification.py

An alternative solution to the problem can look like

To avoid this, a general class could be written and used as base class for image, audio and video.
@FayazRahman FayazRahman added enhancement good first issue labels Mar 19, 2022
@FayazRahman FayazRahman added this to Medium tasks in New Contributor projects Mar 19, 2022
@neel2299
Copy link
Contributor

@neel2299 neel2299 commented Mar 21, 2022

Can I work on this issue ??

@FayazRahman
Copy link
Collaborator Author

@FayazRahman FayazRahman commented Mar 22, 2022

@neel2299 Of course. I'll assign it to you.

@neel2299
Copy link
Contributor

@neel2299 neel2299 commented Mar 29, 2022 •
edited

Hello!! Can we add dependencies to get the video/audio meta data ?? @FayazRahman @farizrahman4u @mikayelh

@farizrahman4u
Copy link
Member

@farizrahman4u farizrahman4u commented Apr 3, 2022 •
edited

Hello!! Can we add dependencies to get the video/audio meta data ?? @FayazRahman @farizrahman4u @mikayelh

No, hub already has all the dependencies required for reading meta data from audio and video.

@farizrahman4u
Copy link
Member

@farizrahman4u farizrahman4u commented Apr 3, 2022 •
edited

Also note that we are switching from miniaudio to PyaAV for audio in this PR : #1576 , so it would be ideal if you base your PR against it.

@vaibhavjack11
Copy link

@vaibhavjack11 vaibhavjack11 commented Apr 4, 2022

I am Interested to work on this issue

@neel2299
Copy link
Contributor

@neel2299 neel2299 commented Apr 5, 2022 •
edited

Noted, Thanks for the heads up. I will check the PR out.

@neel2299 neel2299 mentioned this issue May 1, 2022
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@neel2299 neel2299

Labels
enhancement good first issue
Projects
New Contributor projects
Medium tasks
Milestone
No milestone
Development

No branches or pull requests
4 participants
@farizrahman4u @FayazRahman @vaibhavjack11 @neel2299