Skip to content

add query stats to logs | fix np encoding for logs JSON dump#72

Draft
sureshkumarsrinath wants to merge 1 commit intomainfrom
logging/add-query-time-stats-to-logs
Draft

add query stats to logs | fix np encoding for logs JSON dump#72
sureshkumarsrinath wants to merge 1 commit intomainfrom
logging/add-query-time-stats-to-logs

Conversation

@sureshkumarsrinath
Copy link
Contributor

@sureshkumarsrinath sureshkumarsrinath commented Feb 6, 2026

  • This PR fixes the issue what json dump was not able to comprehend the numpy datatypes produced by FAISS, ranker and embeddings by converting them to primitive type in Python
  • The times of chunk retrieval, ranking, query generation are collected and logged.

TODO: Keep only the required information in the logs and remove the rest of the components in the log JSON object

credits: https://github.com/sukriti112

@RajShah-1
Copy link
Contributor

I think some of these items are also being addressed in: #75.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants