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

Video integration #348

Open
H-Tudor opened this issue Dec 15, 2023 · 1 comment
Open

Video integration #348

H-Tudor opened this issue Dec 15, 2023 · 1 comment

Comments

@H-Tudor
Copy link

H-Tudor commented Dec 15, 2023

The present tool has great capabilities but cannot be used with video files which is a big downside.

The way I see this issue, the application should be able to accept video files like .mp4 and allow you to annotate each frame (with a drop frame button to indicate that the given frame should be ignored at export) and when exporting it should return each frame of the video/s named video_file_name-frame_id in jpg or png format depending on the user selection

How I (a python developer) would approach this issue is to create a branch in the flow when you upload a file and if the file is in a video format, call a python CV2 script (or any other that knows how to process images and videos) and for each frame in each video and creates an new upload file instance named {video_file_name}_{frame_idx} and after processing the videos it shows {x} images uploaded where x = total number of frames across all files, videos and images

Given how to application is constructed, and the fact that the images are not uploaded to the server, I think (correct me if I'm wrong) that they are stored in RAM or other temp memory cache, thus a limit of total images should be in place. Another possible downsize would be image resolution (which is already a formulated issue).

Copy link

👋 Hello @H-Tudor, thank you for your interest in make-sense - free to use online tool for labelling photos! 🏷️

🐞 Bug reports

If you noticed that make-sense is not working properly, please provide us with as much information as possible. To make your life easier, we have prepared a bug report template containing all the relevant details. We know, we ask for a lot... However, please believe that knowing all that extra information - like the type of browser you use or the version of node you have installed - really helps us to solve your problems faster and more efficiently. 😉

💬 Get in touch

If you've been trying to contact us but for some reason we haven't responded to your issue yet, don't hesitate to get back to us on Gitter or Twitter.

💻 Local setup

# clone repository
git clone https://github.com/SkalskiP/make-sense.git

# navigate to main dir
cd make-sense

# install dependencies
npm install

# serve with hot reload at localhost:3000
npm start

To ensure proper functionality of the application locally, an npm 8.x.x and node.js v16.x.x versions are required. More information about this problem is available in the #16 issue.

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

No branches or pull requests

1 participant