cmusatyalab / gabriel

Wearable cognitive assistance using cloudlets
Apache License 2.0
64 stars 41 forks source link

Fix logging problem #21

Closed akindofyoga closed 5 years ago

akindofyoga commented 5 years ago

When Debug.LOG_LEVEL_CONSOLE was set lower than Debug.LOG_LEVEL_FILE, logs at levels less than Debug.LOG_LEVEL_FILE would not get printed to the console. For example, when LOG_LEVEL_FILE was set to logging.ERROR and LOG_LEVEL_CONSOLE was set to logging.INFO, info messages would not get printed to the console. See more details about this issue here.

This change fixes this problem so console messages will be printed based on the value of When Debug.LOG_LEVEL_CONSOLE and file messages will be printed based on the value of Debug.LOG_LEVEL_FILE.

I also fixed some minor issues that I will highlight with inline comments on the pull request.