Most of the Advisor classes lack logging. It is difficult to verify whether the advisor was invoked.
Specifically, ### QuestionAnswerAdvisor, VectorStoreChatMemoryAdvisor.
Would it be nice if SimpleLoggerAdvisor included some insights from other advisors as well?
Comment From: sunyuhan1998
Would it be nice if SimpleLoggerAdvisor included some insights from other advisors as well?
Can you elaborate a bit more? For example, what other information would you like to see in the logs generated by SimpleLoggerAdvisor
?
Comment From: AmritSDutta
I was thinking, can it log the details of the CallAdvisorChain involved? may be the output of advisors as TRACE-level logs. At least QuestionAnswerAdvisor, VectorStoreChatMemoryAdvisor should include some DEBUG logs . @sunyuhan1998