Skip to content

Conversation

@cpyle0819
Copy link
Contributor

No description provided.

@cpyle0819 cpyle0819 requested a review from DavidSouther April 24, 2025 18:21
@cpyle0819 cpyle0819 self-assigned this Apr 24, 2025
@cpyle0819 cpyle0819 force-pushed the corepyle/tributary-snippet-summarizer branch from 9668adb to ebf1fc3 Compare April 25, 2025 17:58
@cpyle0819 cpyle0819 added the Enhancement A general update to the code base. label Apr 25, 2025
@cpyle0819 cpyle0819 force-pushed the corepyle/tributary-snippet-summarizer branch from ebf1fc3 to f9ee53c Compare April 25, 2025 18:23
@cpyle0819 cpyle0819 force-pushed the corepyle/tributary-snippet-summarizer branch from f9ee53c to 9e29cd0 Compare April 25, 2025 18:26
Copy link
Contributor

@DavidSouther DavidSouther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great first pass - let's merge this, and then rewrite with a bit more visibility by doing the three-phase write / ailly / read pass. Yes, it means Ailly will need to handle the bedrock calls that we have in here, but, also means we can keep all the LLM code in Ailly.

@cpyle0819 cpyle0819 force-pushed the corepyle/tributary-snippet-summarizer branch from d6fb7bb to 402aaf2 Compare April 28, 2025 17:01
@cpyle0819 cpyle0819 merged commit 8e3321f into main Apr 28, 2025
2 checks passed
@cpyle0819 cpyle0819 deleted the corepyle/tributary-snippet-summarizer branch April 28, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement A general update to the code base.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants