Re: [PATCH] tests/functional: fix pylint false positives for cv2 module

On 17/08/2026 09.57, Marc-André Lureau wrote:
> Hi
> 
> On Mon, Aug 17, 2026 at 11:19 AM Thomas Huth <[email protected]> wrote:
>>
>> On 15/08/2026 09.24, [email protected] wrote:
>>> From: Marc-André Lureau <[email protected]>
>>>
>>> Add generated-members=cv2.* to pylintrc so pylint skips member
>>> checking on the cv2 C extension module, whose members are not
>>> visible to static analysis.
>>>
>>> Silence:
>>> 2026-08-15 10:42:08,710 - INFO: qemu-test.test_pylint Checking files in /home/elmarco/src/qemu.qom-qapi/tests/functional/arm with pylint
>>> 2026-08-15 10:42:10,941 - ERROR: qemu-test.test_pylint "/home/elmarco/src/qemu.qom-qapi/tests/functional/arm/test_integratorcp.py:83: E1101: Module 'cv2' has no 'imread' member (no-member)"
>>
>> Alex recently already sent a patch for that error here:
>>
>>    https://gitlab.com/qemu-project/qemu/-/commit/dd14a21b9dc1
>>
>> Didn't that help to silence the message in your case, too?
> 
> Ah I missed that, but it doesn't help:
> 
> pylint --rcfile ../tests/functional/pylintrc ../tests/functional/arm/
> ************* Module test_integratorcp
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:83:25:
> E1101: Module 'cv2' has no 'imread' member (no-member)
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:84:26:
> E1101: Module 'cv2' has no 'cvtColor' member (no-member)
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:84:55:
> E1101: Module 'cv2' has no 'COLOR_BGR2GRAY' member (no-member)
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:85:17:
> E1101: Module 'cv2' has no 'matchTemplate' member (no-member)
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:85:52:
> E1101: Module 'cv2' has no 'imread' member (no-member)
> /home/elmarco/src/qemu.nohmp/tests/functional/arm/test_integratorcp.py:86:35:
> E1101: Module 'cv2' has no 'TM_CCOEFF_NORMED' member (no-member)
> 
> My understanding:
> 
> I1101 (c-extension-no-member) - disable: silence if pylint can't
> introspect the module
> 
> E1101 (no-member) - but we need to tell pylint to trust any member
> access on cv2 if it can
Ok, but could you then add it to the "disable=..." list instead, so that we 
disable this for all other modules, too? ... otherwise we'll be busy adding 
an entry for all other modules where this pops up in the course of time.

  Thanks,
   Thomas