Wrong Logs are back

What commit version of aidge do you use

• aidge_core: 0.4.0
• aidge_quantization: 0.3.0

Problem description

We previously noticed that the log level was not shared between python and cpp files. This commit solved this issue.

However, with the latest modifications we noticed that the Logs are no longer shared between modules. For instance, when running a python file which sets the Log using the code below :

parser.add_argument("--test", action="store_true", help="Export in test mode")
parser.add_argument(
    '-v', '--verbose',
    action='count',
    default=0,
    help = (
        "Set the verbosity level of the console output."
        "Use -v to increase verbosity, with the following levels in ascending ordern"
        "default WARN - Only warnings and higher (WARN, ERROR, FATAL) are displayed.n"
        "-v NOTICE - Notices and higher (NOTICE, WARN, ERROR, FATAL) are displayed.n"
        "-vv INFO - Informational messages and higher (INFO, NOTICE, WARN, ERROR, FATAL) are displayed.n"
        "-vvv DEBUG - All messages, including debug information, are displayed.n"
        "Available levels in descending order of severityn"
        "DEBUG < INFO < NOTICE < WARN < ERROR < FATAL."
    )
)
args = parser.parse_args()

# Setting Aidge verbose level
if args.verbose == 0:
    aidge_core.Log.set_console_level(aidge_core.Level.Error)
elif args.verbose == 1:
    aidge_core.Log.set_console_level(aidge_core.Level.Notice)
elif args.verbose == 2:
    aidge_core.Log.set_console_level(aidge_core.Level.Info)
elif args.verbose >= 3:
    aidge_core.Log.set_console_level(aidge_core.Level.Debug) 

... it appears that the Log level is not set correctly for the quantization module, as running the PTQ will show all kinds of Logs.

In the output below, the log has been set to Error :

assigned to @pineapple, @mick94, @cmoineau, @axelfarr, and @bhalimi

Note that the Log level is correctly set for aidge_core, however the Logs are displayed while running the quantize_network function.

This has to do with with PyCapsules management and common static variables not handled correctly by modules

added Bug 🐛 label

added LanguagePyBind label

added Important ❗ label

mentioned in issue #224 (closed)

mentioned in commit daa4ef53

mentioned in commit 9b70101b

mentioned in merge request !326 (merged)

mentioned in commit c5a91f3b

Fixed by merged MR

closed

mentioned in commit 094e3628

mentioned in commit a18df16c

mentioned in commit maab05/aidge_core@8611d845