Skip to content

Logging setup and usage #16

Description

@gabemery

Current code uses logging in a limited way. It is in particular not using standard declaration of a logger which would allow to configure the logging level more easily.
An implementation of such a functionality is available in PR #15 .

A more in depth usage, could also be considered. Current messages in INFO level would probably be more adapted to DEBUG level. The possibility to implement more levels, e.g. MOREINFO between INFO and DEBUG could also be considered but this is not standard.

Also, we should decide if the log level should be configurable as an input to the classes (like ctapipe tools) or externally with a line like this logging.getLogger('acceptance_modelisation').setLevel(logging.INFO).

List of open items :

  • Implement loggers in all file and remove direct logging.log usage (also remove any 'print') (Merged PR Generalise logging usage #18 )
  • Adjust messages log level
  • Decide on log_levels to be used (custom for more verbose output? or only use debug for this purpose)
  • Decide on the strategy for log_level selection at execution time
  • Test more the setup functions and fix unwanted behaviours

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestquestionFurther information is requested

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions