CI (Ubuntu ARM64 x2) showed the Python test suite aborting the whole
process with exit code 134: on a Qt-enabled build, namedWindow() tried
the "xcb" platform plugin, found no display (the Python test step,
unlike the C++ gtest step, does not run under xvfb-run), and Qt
aborted the process instead of raising a catchable exception -- so the
existing try/except around namedWindow never got a chance to run.
Fix: set QT_QPA_PLATFORM=offscreen (only if not already set) before
any GUI call. "offscreen" is the platform plugin Qt ships specifically
for headless/CI use, and is listed as available in the failing CI
job's own log output.
Verified locally: built opencv_python3 with WITH_QT=ON (real Qt6, not
the WITH_QT=OFF build used in the initial verification), and confirmed
that in a clean environment (no DISPLAY, no QT_QPA_PLATFORM preset),
namedWindow + saveWindowParameters + loadWindowParameters all succeed
with this fix, where they would previously depend on a native display
backend being reachable.
Both functions persist/restore a window's size, position, flags,
trackbar values, and zoom/pan state (Qt highgui backend only), take
only a plain String argument, and were already fully documented -- but
were declared CV_EXPORTS instead of CV_EXPORTS_W, so they were
unreachable from Python.
Sibling Qt-only functions in the same header (displayOverlay,
displayStatusBar) already follow the CV_EXPORTS_W + runtime
"not implemented without Qt" convention this change follows.
Adds a Python test that exercises namedWindow + save/loadWindowParameters
and gracefully skips when no GUI backend or no Qt support is present,
consistent with how other environment-dependent tests in this suite
(e.g. CUDA) skip rather than fail.
Verified locally: built opencv_python3 from this branch (WITH_QT=OFF)
and confirmed cv2.saveWindowParameters / cv2.loadWindowParameters are
present on the built module and correctly raise the expected
"compiled without QT support" error, proving the binding reaches the
C++ layer end-to-end.
Replace System.exit(-1) with exceptions in HighGui.java (#28696) #28699
## Summary
Library code should never call System.exit() as it kills the entire JVM. Replaced all 3 instances with appropriate exceptions.
Closes#28696
## Changes
- `imshow()` with empty image: `System.exit(-1)` -> `throw new IllegalArgumentException("Image is empty")`
- `waitKey()` with no windows: `System.exit(-1)` -> `throw new IllegalStateException("No windows created. Call imshow() first")`
- `waitKey()` with null window image: `System.exit(-1)` -> `throw new IllegalStateException("No image set for window: ... Call imshow() first")`
- `InterruptedException` catch: `printStackTrace()` -> `Thread.currentThread().interrupt()`
The objective is to:
*Reduce greatly the number of lines of code in the Java codes;
*Make it easy for Java users to add a trackbar and show the results;
*Get the code more similar between C++, Java and Python, making the tutorials more uniform.
Thebinary110
Alexander Smorkalov
ZIHAN DAI
Maksim Shabunin
Corentin Noël
sepperliu
Alexander Alekhin
Alexander Alekhin
Vitaly Tuzov
tribta
abratchik