Skip to content

Update usage of aidge_core.Log

Cyril Moineau requested to merge UseLog into dev

Current ONNX mix the use of print statement and aidge_core.Log.

I cleaned files by using aidge_core.Log as much as possible.

Cases where I did not modify:

  • Unit test: always use print.
  • Verbose argument: Verbose argument should be removed from aidge_onnx function in favor of only the Log.notice function. However this changes API and I prefer to discuss it in an other issue.
  • Function that explicitly need to print stuff: like coverage report. the expected behavior of these functions is to print and thus does not use the Log system.
Edited by Cyril Moineau

Merge request reports

Loading