Skip to content

JavaScript: add TrackerMIL #21032

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

Merged
merged 1 commit into from
Nov 23, 2021
Merged

JavaScript: add TrackerMIL #21032

merged 1 commit into from
Nov 23, 2021

Conversation

alalek
Copy link
Member

@alalek alalek commented Nov 10, 2021

To verify that Tracker wrapper works.

Blocked by #21045

force_builders_only=docs,Custom
buildworker:Docs=linux-4,linux-6
build_image:Docs=docs-js:18.04
build_image:Custom=javascript
buildworker:Custom=linux-4,linux-6

@alalek
Copy link
Member Author

alalek commented Nov 12, 2021

@murata- Thank you for looking on this!

There is another problem with handling of output Rect arguments.
Currently this feature is blocked by #21045

@murata-
Copy link

murata- commented Nov 13, 2021

@alalek Thank you for your reply.
I pulled a change for # 21045, but tracking didn't work.
TestRectOutput The test failed.

@alalek
Copy link
Member Author

alalek commented Nov 13, 2021

There is no fix yet. #21045 is just a reproducer.

@alalek
Copy link
Member Author

alalek commented Nov 18, 2021

@murata- Added workaround for the output rect problem, please take a look (API is updated, see test code).

@murata-
Copy link

murata- commented Nov 21, 2021

@alalek
Thank you very much.
It worked :'-) :’‐D

After building, change bin/opencv.js (line 34)
< return cv.ready

return cv

@alalek
Copy link
Member Author

alalek commented Nov 23, 2021

👍

@opencv-pushbot opencv-pushbot merged commit 12c1e1d into opencv:4.x Nov 23, 2021
@alalek alalek mentioned this pull request Dec 30, 2021
@alalek alalek mentioned this pull request Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants