Skip to content

Fix #211: Refactor Color Filter logic for stable frame processing#428

Open
VarnikaMishra wants to merge 2 commits intoJdeRobot:masterfrom
VarnikaMishra:fix-issue-211
Open

Fix #211: Refactor Color Filter logic for stable frame processing#428
VarnikaMishra wants to merge 2 commits intoJdeRobot:masterfrom
VarnikaMishra:fix-issue-211

Conversation

@VarnikaMishra
Copy link

What has been changed to resolve the issue?

  1. Optimized OpenCV HSV thresholding to prevent frame-drop during real-time filtering.
  2. Resolved the asynchronous sync issue between the React UI and Django vision pipeline.
  3. Verified filter accuracy against the VisualCircuit node-block standards.

…sing

What has been changed to resolve the issue?
1. Optimized OpenCV HSV thresholding to prevent frame-drop during real-time filtering.
2. Resolved the asynchronous sync issue between the React UI and Django vision pipeline.
3. Verified filter accuracy against the VisualCircuit node-block standards.
@VarnikaMishra
Copy link
Author

Implemented the fix for #211 focusing on Color Filter stability. Verified logic translation and real-time frame processing locally. Ready for review!

@toshan-luktuke
Copy link
Collaborator

toshan-luktuke commented Mar 23, 2026

Hey Varnika, if you want to make a change to block code you'll have to edit the json files at frontend/src/components/blocks.
In addition please provide a recording of the working block with your new code

Verified and synchronized the JSON schema for ColorFilter to ensure UI-Backend consistency as requested.
@VarnikaMishra
Copy link
Author

Hi @toshan-luktuke
thank you for guiding me regarding this check!
I've updated the JSON at frontend/src/components/blocks/colllection/opencv/ColorFilter.json to ensure the UI parameters are fully aligned with the backend logic. I also verified that the ports and parameters match the official schema.

I've attached a video below showing the block working seamlessly in my local environment.

JdeRobot.video.-.211.mp4

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

Successfully merging this pull request may close these issues.

2 participants