Closed ryamauchi closed 2 months ago
使用する LLM を名前で初期化するのは便利だと思いますが,その一方で,
LLMFunction
temperature
ベースとなる LLMFunction は LLM インスタンスを渡す仕様にしたうえで,名前から LLMFuncton を初期化する方法(クラスなりメソッドなり)を別途用意するほうが見通しが良いように思うのですがいかがでしょうか @fuyu-quant
LLMFuncton
使用する LLM を名前で初期化するのは便利だと思いますが,その一方で,
LLMFunction
クラスの中身を書き換える必要があるtemperature
等の設定を外部から設定する方法が無くなってしまっている といった問題があるかと思います.ベースとなる
LLMFunction
は LLM インスタンスを渡す仕様にしたうえで,名前からLLMFuncton
を初期化する方法(クラスなりメソッドなり)を別途用意するほうが見通しが良いように思うのですがいかがでしょうか @fuyu-quant