AgentOps-AI / tokencost

Easy token price estimates for 400+ LLMs. TokenOps.
https://agentops.ai
MIT License
1.48k stars 61 forks source link

Use module logger instead of root logger #63

Closed joaonadkarni closed 4 months ago

joaonadkarni commented 4 months ago

Currently this package is logging to the root logger which makes it harder to manage its logging. For a more details on why the root logger should be avoided you can also check the reasons mentioned in the 1st point here

In this change I take the logging out of the root logger and have a logger for the module.

joaonadkarni commented 4 months ago

@areibman when you have time looking for a review here

areibman commented 4 months ago

Looks great-- thanks for waiting @joaonadkarni !