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.

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

[JS/PY/C#] Tools, Sequence + Monologue Augmentation with Streaming #2098

Open
Tracked by #1492
lilyydu opened this issue Oct 8, 2024 · 2 comments · Fixed by #2214
Open
Tracked by #1492

[JS/PY/C#] Tools, Sequence + Monologue Augmentation with Streaming #2098

lilyydu opened this issue Oct 8, 2024 · 2 comments · Fixed by #2214
Labels
JS & dotnet & Python Change or fix must apply to all three programming languages medium tshirt size medium (5-10 days) P1

Comments

@lilyydu
Copy link
Contributor

lilyydu commented Oct 8, 2024

No description provided.

@lilyydu lilyydu mentioned this issue Oct 8, 2024
Open
@lilyydu lilyydu added P0 JS & dotnet & Python Change or fix must apply to all three programming languages small tshirt size small (1-4 days) labels Oct 8, 2024
@asith-w
Copy link

asith-w commented Oct 9, 2024 •
edited
Loading

Hello

Is this streaming feature now working (dotnet/c#) ?

@lilyydu
Copy link
Contributor Author

lilyydu commented Oct 9, 2024

Hey @asith-w,

This feature is currently in the rollout phase and is available only to specific tenants. Our team is actively working on enabling this feature fully on Teams and across all languages in the SDK.

Once the feature is generally available, we will make a public announcement here:
#2069

@lilyydu lilyydu added P1 medium tshirt size medium (5-10 days) and removed P0 small tshirt size small (1-4 days) labels Nov 8, 2024
@lilyydu lilyydu changed the title [JS/PY/C#] Tools + Stream Enablement [JS/PY/C#] Tools, Sequence + Monologue Augmentation with Streaming Nov 15, 2024
@corinagum corinagum mentioned this issue Nov 21, 2024
Merged
@Stevenic Stevenic mentioned this issue Dec 3, 2024
Merged
1 task
singhk97 added a commit that referenced this issue Dec 9, 2024
@Stevenic @singhk97
@lilyydu @dependabot @corinagum @yiqing-zhao @aacebo
[C#] feat: Implemented basic changes needed to support streaming tool…
2ba12e3 
… calls (#2214)

## Linked issues

closes: #2098

## Details

Port the streaming support for tools augmentation to C#.

#### Change details

- Added StreamingChatToolCallsBuilder and SequenceBuilder helper
classes.
- Updated OpenAIModel class to process tool call chunks and return them
as a message with ActionCalls.
- Updated the LLMClient class to pass through the ActionCalls without
closing the streaming response session.


**code snippets**:

**screenshots**:

## Attestation Checklist

- [ ] My code follows the style guidelines of this project

- I have checked for/fixed spelling, linting, and other errors
- I have commented my code for clarity
- I have made corresponding changes to the documentation (updating the
doc strings in the code is sufficient)
- My changes generate no new warnings
- I have added tests that validates my changes, and provides sufficient
test coverage. I have tested with:
  - Local testing
  - E2E testing in Teams
- New and existing unit tests pass locally with my changes

### Additional information

> Feel free to add other relevant information below

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavin <[email protected]>
Co-authored-by: Lily Du <[email protected]>
Co-authored-by: lilydu <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Corina <[email protected]>
Co-authored-by: Yiqing Zhao <[email protected]>
Co-authored-by: Yiqing Zhao <[email protected]>
Co-authored-by: Alex Acebo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
JS & dotnet & Python Change or fix must apply to all three programming languages medium tshirt size medium (5-10 days) P1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[C#] feat: Implemented basic changes needed to support streaming tool calls Stevenic/teams-ai
2 participants
@asith-w @lilyydu