search

arangoml / networkx-adapter

The NetworkX adapter for ArangoDB
Apache License 2.0
24 stars 8 forks source link

Clean up logging #88

Closed aMahanna closed 2 years ago

aMahanna commented 2 years ago

Following #86, it would be best to clean up the logging statements such that:

  1. Current logging.INFO are moved "up" to another logging level (most likely will have to create a custom level for that)
  2. Current non-node-level & non-edge-level are moved "up" to the logging.INFO level. Furthermore, more of these logging statements should be added around adapter method operations (I.e the user should be able to see a stack trace of the method calls)
  3. Node-level & Edge-level logs remain in the logging.DEBUG level
aMahanna commented 2 years ago

One thing to also consider (perhaps in replacement of node/edge-level debug logs), is to use a progress bar package such as https://pypi.org/project/tqdm/