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.

Sign up for GitHub

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

Jump to bottom

feat(2658): introduce Adapter #2659

Merged
merged 13 commits into from
Aug 12, 2024
Merged

feat(2658): introduce Adapter #2659

merged 13 commits into from
Aug 12, 2024

Conversation

ssddOnTop
Copy link
Member

Issue Reference(s):
Fixes #2658
/claim 2658

@ssddOnTop ssddOnTop marked this pull request as ready for review August 11, 2024 09:11
@algora-pbc algora-pbc bot mentioned this pull request Aug 11, 2024
Closed
@github-actions github-actions bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Aug 11, 2024
tusharmath
tusharmath reviewed Aug 11, 2024
View reviewed changes
src/cli/llm/adapter.rs Outdated Show resolved Hide resolved
tusharmath
tusharmath reviewed Aug 11, 2024
View reviewed changes
src/cli/llm/wizard.rs Outdated Show resolved Hide resolved
@codecov Codecov
Copy link

codecov bot commented Aug 11, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Project coverage is 86.46%. Comparing base (5d3f1ad) to head (240cd66).

Files Patch % Lines
src/cli/llm/wizard.rs 0.00% 8 Missing ⚠️
src/cli/generator/generator.rs 0.00% 6 Missing ⚠️
src/cli/llm/infer_type_name.rs 0.00% 6 Missing ⚠️
src/cli/llm/model.rs 0.00% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2659      +/-   ##
==========================================
- Coverage   86.53%   86.46%   -0.08%     
==========================================
  Files         252      253       +1     
  Lines       24353    24373      +20     
==========================================
  Hits        21073    21073              
- Misses       3280     3300      +20

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ssddOnTop ssddOnTop changed the title feat(llm): introduce Adapter feat(llm): introduce Adapter #2658 Aug 12, 2024
@ssddOnTop ssddOnTop changed the title feat(llm): introduce Adapter #2658 feat(llm): introduce Adapter 2658 Aug 12, 2024
@ssddOnTop ssddOnTop changed the title feat(llm): introduce Adapter 2658 feat(2658, llm): introduce Adapter Aug 12, 2024
@ssddOnTop ssddOnTop changed the title feat(2658, llm): introduce Adapter feat(2658): introduce Adapter Aug 12, 2024
tusharmath
tusharmath reviewed Aug 12, 2024
View reviewed changes
src/cli/llm/model.rs Outdated Show resolved Hide resolved
@tusharmath tusharmath enabled auto-merge (squash) August 12, 2024 14:33
@tusharmath tusharmath merged commit e1d7f69 into main Aug 12, 2024
29 of 31 checks passed
@tusharmath tusharmath deleted the feat/llm-adapter branch August 12, 2024 14:58
@tusharmath tusharmath mentioned this pull request Aug 12, 2024
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tusharmath tusharmath tusharmath approved these changes

Assignees
No one assigned
Labels
🙋 Bounty claim type: feature Brand new functionality, features, pages, workflows, endpoints, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update wizard to programatically control the LLM adapter and model
2 participants
@ssddOnTop @tusharmath