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

CAAFE: Add Bedrock Support #76

Merged
merged 2 commits into from
Nov 1, 2024
Merged

CAAFE: Add Bedrock Support #76

merged 2 commits into from
Nov 1, 2024

Conversation

AnirudhDagar
Copy link
Collaborator

Issue #, if available:
Add Bedrock support to CAAFE. This PR first requires merging AnirudhDagar/CAAFE#1

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@AnirudhDagar AnirudhDagar marked this pull request as ready for review November 1, 2024 00:45
@AnirudhDagar
Copy link
Collaborator Author

AnirudhDagar commented Nov 1, 2024

I've merged the bedrock support PR in the fork of CAAFE (AnirudhDagar/CAAFE#1) we are using in AGA. This PR is now ready for review.

@AnirudhDagar AnirudhDagar requested review from boranhan, canerturkmen and FANGAreNotGnu and removed request for canerturkmen November 1, 2024 00:46
boranhan
boranhan previously approved these changes Nov 1, 2024
@boranhan boranhan dismissed their stale review November 1, 2024 02:41

misclicked

@boranhan
Copy link
Collaborator

boranhan commented Nov 1, 2024

Do you have a run ID?

@AnirudhDagar
Copy link
Collaborator Author

I'll now initiate a run for this and share once it is done.

@AnirudhDagar
Copy link
Collaborator Author

run ID: 589 shows that bedrock does the expected job w CAAFE.

@boranhan
Copy link
Collaborator

boranhan commented Nov 1, 2024

what is the final result of that run?

@boranhan
Copy link
Collaborator

boranhan commented Nov 1, 2024

saw this results offline, LGTM

@AnirudhDagar AnirudhDagar merged commit bdfe008 into main Nov 1, 2024
1 check passed
@AnirudhDagar AnirudhDagar deleted the add_bedrock_support_caafe branch November 5, 2024 17:41
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.

2 participants