Replies: 1 comment
-
|
hey @mattijsdp - haven't forgot about this! i think a rough plan to have a just thinking out loud, am open to reviewing sketch PRs here - otherwise I'll take a stab at this when I can |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
This is a discussion for ways in which Marvin can be integrated with Prefect. After ControlFlow was 'absorbed' into Marvin 3.0, the implicit dependence on Prefect no longer exists. This is great for more general adoption and means you can use any OTEL observability platform like the pydantic team's, logfire. I think there is a sizable audience (including myself) however, who enjoyed the automatic observability ControlFlow provided through Prefect.
What did this look like?
For those not familiar, if you ran ControlFlow flow then it automatically created three main types of tasks:
Flows (i.e. Threads) were also Prefect flows and so would show up as subflows etc.
There might have been more complex things but I think that was the gist.
How could this be done with Marvin?
pydantic-ai's Tool run function: marvin/examples/slackbot/wrap.pypydantic_ai's instrumentation uses OpenTelemetry so perhaps we could create a Prefect basedTracerProviderandEventLoggerProviderpydantic-ai docs?Personal view
I think a Prefect extra/setting would work very nicely and make it very easy to get started quickly. Perhaps this is due to my lack of familiarity with observability tools but I haven't found an open-source one that is as easy as running
prefect server start --background. The default markdown artifacts that ControlFlow had were also generic enough to work for most tools and llm calls but detailed enough to be useful (having the entire LLM payload there is very useful when you are first starting out). For more advanced/production use cases, I think you would either move to custom Prefect 'wrapping' or an actual observability tool.Beta Was this translation helpful? Give feedback.
All reactions