Llama2 の削除

[tbrand]

変更内容の説明

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

チェック項目

• npm run lint を実行した
• 関連するドキュメントを修正した
• 手元の環境で動作確認済み

関連する Issue

#599

[kazuhitogo]

すっごく細かいんですが、

generative-ai-use-cases-jp/packages/types/src/text.d.ts

Line 71 in 4992bc8

// Llama2

にある Llama"2" の 2 をこの際消すのはどうでしょうか。
他はLGTMです。

[tbrand]

そちらを含めて、Llama2 という記述はあと 4 箇所あります。SageMaker 経由で利用される可能性があるかなと思ってそれらは残してあるのですが、これらも不要ですか？

• https://github.com/aws-samples/generative-ai-use-cases-jp/blob/remove-llama2/packages/types/src/text.d.ts#L71
• https://github.com/aws-samples/generative-ai-use-cases-jp/blob/remove-llama2/packages/types/src/text.d.ts#L169
• https://github.com/aws-samples/generative-ai-use-cases-jp/blob/remove-llama2/packages/cdk/lambda/utils/models.ts#L47
• https://github.com/aws-samples/generative-ai-use-cases-jp/blob/remove-llama2/packages/cdk/lambda/utils/models.ts#L928-L929

[kazuhitogo]

ありがとうございます。（他にもあったのを見つけてくださってありがとうございます）
SageMaker 経由は残したいですが、2024/11 現在で Llama "2" を使うケースは減っており、3系を使うことが多い＆汎用性が高い"llama"という表現で統一する、でいかがでしょうか。

[tbrand]

こちらで対応の認識あっていますでしょうか？最新コミットを確認していただけますと 🙏
fd6fdc9

[kazuhitogo]

LGTMです！