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

Adds function to get state to tracker #48

Merged
merged 2 commits into from
Mar 1, 2024
Merged

Conversation

skrawcz
Copy link
Contributor

@skrawcz skrawcz commented Feb 29, 2024

So that we can pull the last state given an ID from the UI.

Updates examples and adds a new one that mirrors the LCEL example with the tool calling. This variant works. Where as the Hamilton DAG also doing tool calling somehow doesn't quite work as well. Still digging in there. Need a run diff tool now... or a way to compare runs...

burr/tracking/client.py Outdated Show resolved Hide resolved
Copy link
Contributor

@elijahbenizzy elijahbenizzy left a comment

Choose a reason for hiding this comment

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

Add:

  1. Section in concepts about saving/loading state
  2. README for the example

burr/core/application.py Outdated Show resolved Hide resolved
burr/core/application.py Outdated Show resolved Hide resolved
burr/tracking/client.py Outdated Show resolved Hide resolved
burr/tracking/client.py Outdated Show resolved Hide resolved
burr/tracking/client.py Outdated Show resolved Hide resolved
So that we can pull the last state given an ID from the UI.

Updates examples and adds a new one that mirrors the
LCEL example with the tool calling. This variant works.
Where as the Hamilton DAG also doing tool calling somehow doesn't quite work
as well. Still digging in there. Need a run diff tool now... or a way to compare runs...
@skrawcz skrawcz force-pushed the tweak_multi_agent branch from 2a2148f to 15d24cd Compare March 1, 2024 00:17
Copy link
Contributor

@elijahbenizzy elijahbenizzy left a comment

Choose a reason for hiding this comment

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

LGTM

@elijahbenizzy elijahbenizzy merged commit c78170e into main Mar 1, 2024
5 checks passed
@elijahbenizzy elijahbenizzy deleted the tweak_multi_agent branch March 1, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants