Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

POC - AWS Bedrock #2510

Draft
wants to merge 66 commits into
base: master
Choose a base branch
from
Draft

POC - AWS Bedrock #2510

wants to merge 66 commits into from

Conversation

rauboti
Copy link
Contributor

@rauboti rauboti commented Oct 14, 2024

Draft PR for POC i AWS Bedrock. PR Dekker funksjonalitet fra steg 1 og 2, som skal:

  • Generere oppsummering av læringsartikkel
  • Generere metabeskrivelse av læringsartikkel
  • Generere kontrollspørsmål fra læringsartikkel
  • Be om omformulering av tekstutdrag

@gunnarvelle
Copy link
Member

I https://github.com/NDLANO/deploy/pull/949 har eg satt verdiene NDLA_AI_SECRET_ID og NDLA_AI_SECRET_KEY i secrets.tf i test basert på brukeren som er oppretta der. No forventer BedrockClient at verdiene heiter AWS_ACCESS_KEY_ID og AWS_SECRET_ACCESS_KEY siden dei ikkje er definert eksplisitt.
Kan fikses med noko sånt:

  const secretId = getEnvironmentVariabel("NDLA_AI_SECRET_ID", "");
  const secretKey = getEnvironmentVariabel("NDLA_AI_SECRET_KEY", "");
  const client = new BedrockRuntimeClient({
    region: modelRegion,
    credentials: { accessKeyId: secretId, secretAccessKey: secretKey },

rauboti and others added 30 commits October 28, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants