aws-samples / generative-ai-use-cases-jp

すぐに業務活用できるビジネスユースケース集付きの安全な生成AIアプリ実装
MIT No Attribution
695 stars 126 forks source link

Rag Knowledge Base の検索対象をS3プレフィックスごとに選択可能に #705

Closed xianglishan closed 1 week ago

xianglishan commented 2 weeks ago

変更内容の説明

変更内容を詳細に説明して下さい。 既存ユーザーへの影響がある場合 (互換性・デグレ・破壊的変更など) は必ず説明に含めてください。

変更は以下

既存ユーザーでRagKnowledgeBaseユースケース有効化している場合に影響あり

チェック項目

関連する Issue

関連する Issue を可能な限り挙げてください。

(初プルリクなので変なことしてたらすみません…)

tbrand commented 2 weeks ago

初 PR ありがとうございます!

S3 の prefix を選択できるようにするという機能は有用である一方、全利用ユーザーに対して共通で必要かというと判断が難しいです。そのため、こちらの PR をマージすることは難しいです。申し訳ございません!

まずは Issue 化していただき、仕様について議論させていただくとスムーズに進められると思いました!

また、初 PR ということですので、僭越ながらいくつか diff にコメントさせていただきます。 今後の PR のために、参考にしていただけると幸いです。引き続きよろしくお願いいたします。

xianglishan commented 1 week ago

@tbrand レビューいただきありがとうございます。 初めてでしたので試行錯誤しながらという感じでしたが、コメントもいれていただいて大変勉強になります。 今後のためにもIssue、プルリクいろいろ引き続き見てみます! 改めてありがとうございました! このプルリクはクローズしておきます