Skip to content

Refactor managed llm samples #384

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
Merged
merged 14 commits into from
May 23, 2025
Merged

Refactor managed llm samples #384

merged 14 commits into from
May 23, 2025

Conversation

jordanstephens
Copy link
Member

@jordanstephens jordanstephens commented May 22, 2025 •
edited by github-actions bot
Loading

  • mount ./app for development
  • avoid navigating to a new page on form submission
  • use a static index.html
  • move javascript to a static file
  • factor out a new get_llm_response function

Samples Checklist

✅ All good!

commit111
commit111 reviewed May 22, 2025
View reviewed changes
@jordanstephens @commit111
Update comment
0b0e24d 
Co-authored-by: Linda L. <llee008844@gmail.com>
commit111
commit111 reviewed May 22, 2025
View reviewed changes
jordanstephens and others added 2 commits May 22, 2025 15:56
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 22, 2025 23:28 — with GitHub Actions Failure
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 22, 2025 23:43 — with GitHub Actions Failure
@commit111 commit111 had a problem deploying to deploy-changed-samples May 22, 2025 23:56 — with GitHub Actions Failure
@commit111 commit111 had a problem deploying to deploy-changed-samples May 23, 2025 00:00 — with GitHub Actions Failure
jordanstephens
jordanstephens commented May 23, 2025
View reviewed changes
jordanstephens
jordanstephens commented May 23, 2025
View reviewed changes
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 23, 2025 00:21 — with GitHub Actions Error
@jordanstephens jordanstephens force-pushed the jordan/refactor-managed-llm-samples branch from 9abfcfb to 16a9536 Compare May 23, 2025 00:22
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 23, 2025 00:22 — with GitHub Actions Failure
@jordanstephens jordanstephens force-pushed the jordan/refactor-managed-llm-samples branch from 16a9536 to dbe4ab4 Compare May 23, 2025 00:25
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 23, 2025 00:25 — with GitHub Actions Failure
nullfunc
nullfunc reviewed May 23, 2025
View reviewed changes
@jordanstephens jordanstephens had a problem deploying to deploy-changed-samples May 23, 2025 14:26 — with GitHub Actions Failure
@jordanstephens jordanstephens merged commit 3de7f9e into main May 23, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@commit111 commit111 commit111 approved these changes

@nullfunc nullfunc nullfunc 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
@jordanstephens @nullfunc @commit111