New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to python Logging module #124

Closed
jarvisteach opened this Issue Apr 3, 2017 · 1 comment

Comments

Projects
None yet
1 participant
@jarvisteach
Owner

jarvisteach commented Apr 3, 2017

Python provides a built in logging library - https://docs.python.org/3.6/howto/logging.html

Currently appJar provides a warn & debug function, which are used internally, can be muted but not redirected.

Should switch to using logging module, to allow redirection to file, as well as increased logging levels,

appJar should provide wrappers for these, along with a wrapper for the basicConfig function.
BUT, it should also be possible to import logging in the program using appJar, and for the two to work nicely together....

Should also provide a new documentation page - Logging....

@jarvisteach jarvisteach modified the milestone: 0.07 Apr 3, 2017

jarvisteach added a commit that referenced this issue Apr 19, 2017

@jarvisteach

This comment has been minimized.

Show comment
Hide comment
@jarvisteach

jarvisteach Apr 19, 2017

Owner

Switched from printing to logging in self.warn & self.debug.

Now need to go through code and:

  • Remove any prints - replace with DEBUGs
  • Find any places logging is 'paused' - and implement better solution
  • Look into logging exceptions when they're caught
  • Look at how to throw exceptions better - so that they get logged
Owner

jarvisteach commented Apr 19, 2017

Switched from printing to logging in self.warn & self.debug.

Now need to go through code and:

  • Remove any prints - replace with DEBUGs
  • Find any places logging is 'paused' - and implement better solution
  • Look into logging exceptions when they're caught
  • Look at how to throw exceptions better - so that they get logged

jarvisteach added a commit that referenced this issue Apr 19, 2017

Implemented logging pauser (#124)
Implemented a class to use with the `with` command to pause logging.

Went through and replaced instances where the code was paused using the
`__pauseWarn()` and `__resumeWarn()` functions

Testing needs updating

jarvisteach added a commit that referenced this issue Apr 22, 2017

Updates to tooltips (#133)
Can now change/enable/disable tooltips.
Included testing & documentation.

Also, a bit more cleanup for issue #124 - removed commented lines, and
remaining prints.

jarvisteach added a commit that referenced this issue Apr 23, 2017

updates to googleMaps (#136). Including getLocation() function to det…
…ermine currentLocation. Also added <Escape> to AutoEntry (#110). And added new exception() function for logging (#124)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment