Skip to content

bug fix - moviepy version update #942

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.

Sign up for GitHub

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

Merged
ashahba merged 3 commits into from
Nov 22, 2024
Merged

bug fix - moviepy version update #942

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3cf4d52
initial commit with fix
MSCetin37 Nov 21, 2024
19f622a
initial commit with fix
MSCetin37 Nov 21, 2024
59ca466
Merge branch 'main' into fix_for_moviepy_931
MSCetin37 Nov 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion comps/dataprep/multimedia2text/video2audio/video2audio.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
import uuid
from os import path, remove

from moviepy.editor import VideoFileClip
from moviepy import VideoFileClip

# Get the root folder of the current script
root_folder = path.dirname(path.abspath(__file__))
Expand Down
2 changes: 1 addition & 1 deletion comps/dataprep/multimodal/redis/langchain/multimodal_utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
import requests
import webvtt
import whisper
from moviepy.editor import VideoFileClip
from moviepy import VideoFileClip


def create_upload_folder(upload_path):
Expand Down
10 changes: 5 additions & 5 deletions tests/dataprep/test_dataprep_multimedia.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ function start_services() {
fi


docker run -d -p 9099:9099 --name="test-comps-mm-a2t-service" --ipc=host -e http_proxy=$http_proxy -e https_proxy=$https_proxy -e A2T_ENDPOINT=http://$host_ip:7066 opea/a2t:comps
docker run -d -p 9199:9099 --name="test-comps-mm-a2t-service" --ipc=host -e http_proxy=$http_proxy -e https_proxy=$https_proxy -e A2T_ENDPOINT=http://$host_ip:7066 opea/a2t:comps
if [ $? -ne 0 ]; then
echo "opea/a2t service fail to start"
exit 1
Expand Down Expand Up @@ -183,7 +183,7 @@ function validate_microservices() {

# Audio2Text service
validate_services \
"${host_ip}:9099/v1/audio/transcriptions" \
"${host_ip}:9199/v1/audio/transcriptions" \
'"query":"well"' \
"a2t" \
"a2t-service" \
Expand All @@ -192,23 +192,23 @@ function validate_microservices() {
# Video2Audio service
validate_services \
"${host_ip}:7078/v1/video2audio" \
"SUQzBAAAAAAAI1RTU0UAAAAPAAADTGF2ZjU4LjI5LjEwMAAAAAAAAAAAAAAA//tQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAASW5mbwAAAA8AAAAIAAAN3wAtLS0tLS0tLS0tLS1LS0tLS0tLS0tLS0tpaWlpaWlpaWlpaWlph4eHh4eHh4eHh4eHpaWlpaWlpaWlpaWlpcPDw8PDw8PDw8PDw+Hh4eHh4eHh4eHh4eH///////////////8AAAAATGF2YzU4LjU0AAAAAAAAAAAAAAAAJAYwAAAAAAAADd9L18KaAAAAAAAAAAAAAAAAAAAAAP/7kGQAAAMhClSVMEACMOAabaCMAREA" \
"SUQzBAAAAAAAI1RTU0UAAAAPAAADTGF2ZjU4LjI5LjEwMAAAAAAAAAAAAAAA//tQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAASW5mbwAAAA8AAAAIAAAN3wAtLS0tLS0tLS0tLS1LS0tLS0tLS0tLS0tpaWlpaWlpaWlpaWlph4eHh4eHh4eHh4eHpaWlpaWlpaWlpaWlpcPDw8PDw8PDw8PDw+Hh4eHh4eHh4eHh4eH///////////////8AAAAATGF2YzU4LjU0AAAAAAAAAAAAAAA" \
"v2a" \
"v2a-service" \
"{\"byte_str\": \"$(input_data_for_test "video")\"}"

# Docsum Data service - video
validate_services \
"${host_ip}:7079/v1/multimedia2text" \
'"query":"well"' \
'"query":"well' \
"multimedia2text-service" \
"multimedia2text" \
"{\"video\": \"$(input_data_for_test "video")\"}"

# Docsum Data service - audio
validate_services \
"${host_ip}:7079/v1/multimedia2text" \
'"query":"well"' \
'"query":"well' \
"multimedia2text-service" \
"multimedia2text" \
"{\"audio\": \"$(input_data_for_test "audio")\"}"
Expand Down
Loading