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

[BUG] AttributeError in download_video() When Processing VideoInputList #27

Open
movchan74 opened this issue Dec 15, 2023 · 0 comments
Open
Labels
bug Something isn't working wip Work In Progress

Comments

@movchan74
Copy link
Contributor

Bug Description

  • AttributeError encountered in download_video() function, specifically with handling VideoInputList objects. The error states that 'VideoInputList' object has no attribute 'url', indicating an issue with how VideoInputList is processed.

Steps to Reproduce

  1. Start the server with the command: poetry run aana --port 8000 --host 0.0.0.0 --target whisper.
  2. Send a POST request to http://127.0.0.1:8000/video/transcribe with the specified data structure containing the path to the video file.
data = {
    "videos": [
        {"path": "/workspaces/aana_sdk/aana/tests/files/videos/physicsworks.webm"},
    ]
}

url = "http://127.0.0.1:8000/video/transcribe"

response = requests.post(url, data={"body": json.dumps(data)})
response.json()
  1. Observe the error response indicating an AttributeError in download_video() function.

Expected Behavior

  • The server should process the VideoInputList and transcribe the video without encountering an AttributeError.

Actual Behavior

  • The server returns an error response with RayTaskError(AttributeError) indicating that VideoInputList object does not have an 'url' attribute, failing in the download_video() function.

Performance Details (if applicable)

  • N/A

Environment

Additional Notes

The bug was introduced when I removed the automatic convert_input_to_object call in the endpoint generation. My idea is to call convert_input_to_object explicitly in the pipeline when needed. For example, we call it in the download_video function for non-youtube videos. Technically it should already work for VideoInputList because we run download_video with batched=True but since VideoInputList is not a real list it fails. I'm not completely sure what is the bug but it might be something in the Mobius Pipeline in the batched nodes.

@movchan74 movchan74 added the bug Something isn't working label Dec 15, 2023
@ashwinnair14 ashwinnair14 added the wip Work In Progress label Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wip Work In Progress
Projects
None yet
Development

No branches or pull requests

2 participants