search

UpstageAI / aws-examples

1 stars 0 forks source link

Add region map for arn #1

Closed chosh0615 closed 7 months ago

chosh0615 commented 7 months ago

JumpStart 팀에서 요청 온대로 업데이트 합니다.

  1. region map 을 만들어서 arn 사용
  2. notebook output 들 모두 제거
review-notebook-app[bot] commented 7 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

chosh0615 commented 7 months ago

@chosh0615 지금도 문제 없지만 어차피 model_package_name이 공통으로 들어가니 model_package_map의 value에는 model_package_name 제외한 arn만 넢고 model_package_arn 조립할 때 뒤에 붙여주는 게 더 깔끔할 것 같습니다.

그것도 좋은생각인데요, 일단 안전하게 JumpStart 팀에서 가이드로 준 cohere예시 와 동일하게 가보려고 합니다!