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

add _record function forLangtailPrompts and implicit completion flow #18

Closed
wants to merge 9 commits into from

Conversation

capaj
Copy link
Contributor

@capaj capaj commented May 11, 2024

https://linear.app/langtail/issue/LAN-341/sdk-logs-for-proxyless-invocation

  • _record method added
  • implicit flow with the new prompt object
  • docs

this needs https://github.com/langtail/langtail/pull/394 to be merged to work as it uses that endpoint

@capaj capaj requested a review from petrbrzek May 11, 2024 22:20
@capaj
Copy link
Contributor Author

capaj commented May 13, 2024

@rychlis this should be almost ready. Only thing missing is that we need to define a secret here: OPENAI_API_KEY

because the test is now calling open AI API directly.

@capaj capaj requested a review from rychlis May 13, 2024 14:37
@capaj capaj added the enhancement New feature or request label May 13, 2024
@capaj capaj marked this pull request as ready for review May 13, 2024 14:37
@capaj capaj changed the title add _record function forLangtailPrompts add _record function forLangtailPrompts and implicit completion flow May 13, 2024
@rychlis
Copy link
Member

rychlis commented May 24, 2024

We'll do the API a bit differenly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants