UpstageAI / aws-examples

1 stars 0 forks source link

Create Solar_Mini_Chat_Sagemaker_JA_Marketplace .ipynb #6

Closed Rududu-real closed 4 months ago

Rududu-real commented 4 months ago

션께서 요청주신 JA model에 대한 marketplace 실행법입니다. model이름과 설명, 링크가 교체되었고 pip update 등 실제로 돌려봤을때 스무스하지 않던 부분들을 수정했습니다.

확인 부탁드립니다!

review-notebook-app[bot] commented 4 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

namjunjo commented 4 months ago

이렇게 진행하는 것보다 기존 marketplace notebook 에서 model package 명만 추가하면 되지 않을까요? @Rududu-real

Rududu-real commented 4 months ago

저도 그게 구현 상으로는 더 단순하다고는 생각됩니다. 하지만 요 노트북들은 제품 사용 설명서로서 사용되는 것이다보니 서로 다른 제품에 대해서는 다른 노트북을 줘야하지 않나생각했습니다. 문서 서두엔 모델에 대한 소개 등도 포함되어있고요.

앞으로도 비슷하게 chat model이 늘어날수록 한 파일에 설명을 넣기는 힘들지 않을까하는게 제 생각이었는데, 매번 파일 분리하는게 더 복잡하다고 여기신다면 하나로 합치겠습니다!