bug: mistake(?) in python always returns len 1

[kaddkaka]

This list generator is wrapped in 2 levels of brackets, which means the call to len() will always return 1, looks like a mistake.

riscv-dv/pygen/pygen_src/isa/riscv_cov_instr.py

Line 365 in 7e54b67

bit_difference = len([[ones for ones in temp[2:] if ones == '1']])

[MikeOpenHWGroup]

FWIW, I think you are right @kaddkaka. Maybe try bit_difference = temp.count('1')?

[kaddkaka]

I requested a ruff lint rule based on the above code: astral-sh/ruff#19419

What you suggest seems like the intended behavior @MikeOpenHWGroup

I wouldn't know who to supply a change or test/check if it's correct, nor have the time. Hopefully this is more easily fixed by someone who knows the code.