| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
| write_stream: MemoryObjectSendStream[SessionMessage], | ||
| read_timeout_seconds: timedelta | None = None, | ||
| sampling_callback: SamplingFnT | None = None, | ||
| sampling_capabilities: types.SamplingCapability | None = None, |
There was a problem hiding this comment.
thoughts on moving to the bottom of kwargs or keyword only to avoid breaking positional uses of the constructor? I'm probably leaning towards positional keyword only
Sorry, something went wrong.
There was a problem hiding this comment.
Good call! (assuming you meant leaning "keyword" only, and I agree - will update)
Sorry, something went wrong.
There was a problem hiding this comment.
woops yea meant keyword only haha
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
SEP-1577 added tool calling support to sampling (via #1594). This PR adds the necessary client-side support:
FWIW, with these changes we have full agentic sampling wired up in a branch of FastMCP, which is extremely exciting!