mciepluc / cocotb-coverage

Functional Coverage and Constrained Randomization Extensions for Cocotb
BSD 2-Clause "Simplified" License
100 stars 15 forks source link

report_coverage() does not accept a logger object as documented #62

Closed jonpovey closed 2 years ago

jonpovey commented 2 years ago

report_coverage() requires a logger argument documented as:

logger (func) – a logger object.

But it does not accept a "logger object" as described in https://docs.python.org/3/library/logging.html Actually what it wants is a function such as logger.info or print. Suggest changing this documentation line to something less confusing, perhaps something like:

logger (func) - a logging function (e.g. logger.info)

mciepluc commented 2 years ago

thanks, fixed