Skip to content

Update opencv to 4.11.0-1.5.12#1942

Closed
Rylern wants to merge 1 commit intoqupath:mainfrom
Rylern:opencv-update
Closed

Update opencv to 4.11.0-1.5.12#1942
Rylern wants to merge 1 commit intoqupath:mainfrom
Rylern:opencv-update

Conversation

@Rylern
Copy link
Contributor

@Rylern Rylern commented Jul 2, 2025

Update opencv to 4.11.0-1.5.1 to address qupath/qupath-extension-align#15.

This required to add a parameter to opencv_imgproc.GaussianBlur.

@petebankhead petebankhead added this to the v0.7.0 milestone Jul 2, 2025
@petebankhead
Copy link
Member

The AlgorithmHint parameter is a bit disturbing - do you know what the default values is?

Since QuPath should behave similarly across platforms, I think we'd want ALGO_HINT_ACCURATE. Do you see other places that would be impacted by the introduction of this parameter?

(I see it affects the Wand, but that's one case where I guess strict accuracy isn't so important).

@petebankhead
Copy link
Member

Tests pass for me on Mac, but fail on Windows with an UnsatisfiedLinkError related to no jniopencv_core in java.library.path.

So looks like we may be unable to update...

@Rylern
Copy link
Contributor Author

Rylern commented Jul 4, 2025

OK, should I close this PR? Until a next OpenCV update appears

@petebankhead petebankhead marked this pull request as draft July 4, 2025 12:14
@petebankhead
Copy link
Member

I've made it a draft for now, in case we can figure out how to get it working.

@petebankhead
Copy link
Member

Closed because of #1973 and #1994

@Rylern Rylern deleted the opencv-update branch September 23, 2025 15:25
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