[Video] Video Component to extract 1 frame every N seconds

[ShihChun-H]

Issue Description

Current State

• The Video Component minimum FPS is 1 (integer)

Why We Want to Change?

• For a use case like presentation in the video, I do not want to extract 1 frame every second but 1 frame every 10 seconds to save resources and reduce the number of images needed to feed into the next component.

Proposed Change

• Change FPS to accept fractional inputs i.e. 60, 30, 1, 1/2, 1/4, 1/30…

---

Rules for the Component Hackathon

• Each issue will only be assigned to one person/team at a time.
• You can only work on one issue at a time.
• To express interest in an issue, please comment on it and tag @kuroxx, allowing the Instill AI team to assign it to you.
• Ensure you address all feedback and suggestions provided by the Instill AI team.
• If no commits are made within five days, the issue may be reassigned to another contributor.
• Join our Discord to engage in discussions and seek assistance in #hackathon channel. For technical queries, you can tag @chuang8511.

---

Component Contribution Guideline | Documentation | Official Go Tutorial

[samir-advantech]

I am interested in this issue @ShihChun-H

[ShihChun-H]

@samir-advantech Awesome! Let me assign this issue to you. Please make sure to read and follow the rules above 🙌🏻

[ShihChun-H]

Hi @samir-advantech, I'm following up to check on any progress made regarding this issue. Could you please provide an update? Thanks 🙏

[Reckadon]

Hi, @ShihChun-H ! If @samir-advantech is not available or working on this, can I work on this?

[kuroxx]

Hey @samir-advantech, since this issue has not been worked on for over >5 days, I will now assign this issue to @Reckadon 🙏 Thank you for your contribution attempt!

[Reckadon]

hi @kuroxx, thanks for letting me contribute.
I have managed to setup the project but am facing error while building the instill-core. It is related to influxdb
dependency failed to start: container influxdb exited (1)

[kuroxx]

Hey @Reckadon, thanks for raising the issue!

I saw your message on Discord - hope it can be resolved after adjusting the Docker resources : https://www.instill.tech/docs/core/deployment#docker-resource-requirements

[Reckadon]

do we want to accept inputs like 10, 30, 1/5, 1/10 only or, are pure numbers fine? like- 10, 30, 0.2, 0.1, 0.5

accepting both numbers and fractional strings will require the type field in tasks.json to be any.

[chuang8511]

do we want to accept inputs like 10, 30, 1/5, 1/10 only or, are pure numbers fine? like- 10, 30, 0.2, 0.1, 0.5

It seems good that we can support 1/5 or 0.2.

accepting both numbers and fractional strings will require the type field in tasks.json to be any.

Yes, please proceed this. Thanks for confirmation!

[Reckadon]

@chuang8511 but the main issue is can the type field be any, I think only specific types like number or string are supported

[chuang8511]

Oh, I did not notice it.
Could you try to set it as string first?

[Reckadon]

Setting it as string causes an error when u pass in a pure number like 10 from the frontend.
It's because the JSON parser treats it as a number and does not convert it to a number string (10 and not "10").
I think allowing only numbers (integers and floats) and not strings is the only feasible option for now.
Let me know if I can do something to change the behaviour of the Parser.

[Reckadon]

opened a PR to close this issue

[kuroxx]

@chuang8511 @donch1989 please review when you are free 🙏 thank you!