Skip to content
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.

Sign up for GitHub

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

Jump to bottom

A couple of minor adjustments #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

A couple of minor adjustments #19

wants to merge 1 commit into from

Conversation

AlmightyOatmeal
Copy link

  • Restructured code to be consistent with .get() methods.
  • Improved efficiency by reducing nested code blocks in loops.
  • Improved memory consumption by marking objects for deletion so
    Python GC can clean them up (without manually calling GC).
  • Added del and exit methods to try to safely disconnect from
    MongoDB instance.
  • Comment-out the "server not running with --replSet" log spam.
  • Made it prettier.

* Added .get() method to dictionary items to prevent key access errors.
00896f7 
* Restructured code to be consistent with .get() methods.
* Improved efficiency by reducing nested code blocks in loops.
* Improved memory consumption by marking objects for deletion so
  Python GC can clean them up (without manually calling GC).
* Added __del__ and __exit__ methods to try to safely disconnect from
  MongoDB instance.
* Comment-out the "server not running with --replSet" log spam.
* Made it prettier.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dgriff-tools dgriff-tools

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@AlmightyOatmeal @dgriff-tools