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

Add some functions to OpenCV JS API #26387

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

sturkmen72
Copy link
Contributor

@sturkmen72 sturkmen72 commented Oct 30, 2024

Closes : #20133

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake
force_builders=Custom
build_image:Docs=docs-js
Xbuild_image:Custom=javascript
Xbuild_image:Custom=javascript-simd
Xbuild_image:Custom=javascript-simd:3.1.64
build_image:Custom=javascript:3.1.64
buildworker:Custom=linux-4

@asmorkalov asmorkalov added this to the 4.11.0 milestone Oct 30, 2024
@sturkmen72 sturkmen72 force-pushed the js-imgproc branch 3 times, most recently from e87961b to 87c9acd Compare November 6, 2024 12:29
@sturkmen72 sturkmen72 force-pushed the js-imgproc branch 2 times, most recently from cdb4049 to 8a28689 Compare November 11, 2024 23:12
@sturkmen72
Copy link
Contributor Author

sturkmen72 commented Nov 11, 2024

@asmorkalov Do we need to test for all the functions added? Are there any functions that need to be added from the list of unavailable functions?

The functions added

applyColorMap
approxPolyN
arrowedLine
blendLinear
boxPoints
clipLine
convertMaps
createHanningWindow
divSpectrums
drawMarker
findContoursLinkRuns
fitEllipseAMS
fitEllipseDirect
getFontScaleFromHeight
getRectSubPix
HuMoments
intersectConvexConvex
invertAffineTransform
minEnclosingTriangle
preCornerDetect
rotatedRectangleIntersection
sqrBoxFilter
spatialGradient
stackBlur

The functions below are still unavailable

accumulate 
accumulateProduct
accumulateSquare
accumulateWeighted
buildPyramid
cornerEigenValsAndVecs
cornerSubPix
createGeneralizedHoughBallard
createGeneralizedHoughGuil
EMD
getDerivKernels
getGaborKernel
getGaussianKernel
getTextSize
HoughLinesPointSet
phaseCorrelate
pyrMeanShiftFiltering
wrapperEMD

@sturkmen72 sturkmen72 changed the title WIP Add some functions to OpenCV JS API Add some functions to OpenCV JS API Nov 11, 2024
@sturkmen72 sturkmen72 force-pushed the js-imgproc branch 2 times, most recently from d52540c to 8a02fea Compare November 13, 2024 00:42
@asmorkalov
Copy link
Contributor

The functions list is too long. I propose to add test for major functions and non-trivial interfaces.

@sturkmen72 sturkmen72 force-pushed the js-imgproc branch 5 times, most recently from 07bda2d to 310587e Compare November 17, 2024 12:28
@sturkmen72 sturkmen72 force-pushed the js-imgproc branch 4 times, most recently from 39fae2a to f4257a4 Compare November 29, 2024 21:17
applyColorMap
approxPolyN
arrowedLine
blendLinear
boxPoints
clipLine
convertMaps
createHanningWindow
divSpectrums
drawMarker
findContoursLinkRuns
fitEllipseAMS
fitEllipseDirect
getFontScaleFromHeight
getRectSubPix
HuMoments
intersectConvexConvex
invertAffineTransform
minEnclosingTriangle
preCornerDetect
rotatedRectangleIntersection
sqrBoxFilter
spatialGradient
stackBlur
@sturkmen72
Copy link
Contributor Author

@asmorkalov please review. I've added the functions I consider most important and made some adjustments to enhance the code's readability and structure. I'll focus on adding more functions once this PR is merged.

@asmorkalov asmorkalov self-requested a review December 2, 2024 13:59
@asmorkalov asmorkalov self-assigned this Dec 2, 2024
@asmorkalov asmorkalov merged commit ebf3c40 into opencv:4.x Dec 19, 2024
28 of 31 checks passed
@sturkmen72 sturkmen72 deleted the js-imgproc branch December 19, 2024 06:52
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.

opencvJS doesn't support boxPoints API. I hope it does
2 participants