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

[documentation + more apiclient + action workflow to auto test] #18

Merged
merged 28 commits into from
May 20, 2024
Merged

[documentation + more apiclient + action workflow to auto test] #18

merged 28 commits into from
May 20, 2024

Conversation

liyin2015
Copy link
Member

@liyin2015 liyin2015 commented May 17, 2024 •
edited by Sylph-AI
Loading

  • developer notes
  • llm applications intro doc
  • prompt design doc
  • api_client document
  • test case for empty system prompt and different prompts
  • made the api_client more robust, added AnthropicAPIClient and GoogleGenAIClient

@Alleria1809
Copy link
Contributor

I checked the conf.py and index.rst, and the structure looks good.

liyin2015 added 12 commits May 18, 2024 14:20
@liyin2015
add design for major components
83caf87
@liyin2015
add developer notes
ccd6fb3
@liyin2015
clean up the llmretriever and the concept of using Prompt for any str…
0d9abbe 
…ing formating with variables
@liyin2015
moving all sdk depencides such as openai client, groq client, faiss r…
f41c3c8 
…etriever to components, and simplify the import using the import in each __init__
@liyin2015
add input prompt key for generator
fc117e1
@liyin2015
update the prompt to system prompt and we pass two messages or only o…
5fbc9a5 
…ne messaage to different model apis (this might be moved to api layer)
@liyin2015
have better name for api_client
df40419
@liyin2015
make the apiclient more robust to different api, included Anthropic
bd1d0b6
@liyin2015
delete comments
bfc2466
@liyin2015
improved the react agent, taking less steps to reach to the right answer
abddd2f
@liyin2015
clean
b719ea8
@liyin2015
enforce pytest while merging any code to main
c6011c1
Sylph-AI
Sylph-AI approved these changes May 20, 2024
View reviewed changes
Copy link
Collaborator

@Sylph-AI Sylph-AI left a comment

Choose a reason for hiding this comment

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

merge as it is getting too big of a pr

@Sylph-AI Sylph-AI changed the title [wip] [documentation + cookbook checking] [documentation + more apiclient + action workflow to auto test] May 20, 2024
@Sylph-AI Sylph-AI merged commit e2f147f into main May 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sylph-AI Sylph-AI Sylph-AI approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@liyin2015 @Alleria1809 @Sylph-AI