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.

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

feat: use FastAPI logger instead of rich #198

Merged
merged 3 commits into from
Jan 25, 2024
Merged

feat: use FastAPI logger instead of rich #198

merged 3 commits into from
Jan 25, 2024

Conversation

himkt
Copy link
Owner

@himkt himkt commented Jan 24, 2024

Before

INFO:     Uvicorn running on http://0.0.0.0:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [1] using StatReload
INFO:     Started server process [47]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
[15:22:26] INFO     Create tokenizer: tokenizer-janome.user_dictionary_path-None.system_dictionary_path-None.model_path-None.mode-A.dictionary_format-None                                                                                          tokenization.py:50
INFO:     192.168.65.1:47241 - "POST /api/v1/tokenize HTTP/1.1" 200 OK

After

INFO:     Started server process [14994]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://127.0.0.1:8000 (Press CTRL+C to quit)
INFO:     Create tokenizer: tokenizer-janome.user_dictionary_path-None.system_dictionary_path-None.model_path-None.mode-A.dictionary_format-None
INFO:     127.0.0.1:62229 - "POST /api/v1/tokenize HTTP/1.1" 200 OK

@himkt himkt merged commit 21aaadb into main Jan 25, 2024
4 checks passed
@himkt himkt deleted the logging branch January 25, 2024 14:38
@himkt himkt added the dependencies Pull requests that update a dependency file label Jan 25, 2024
@himkt himkt self-assigned this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant