Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Moved to ORTGenAI's IChatClient. #246

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

azchohfi
Copy link
Contributor

No description provided.

@azchohfi azchohfi marked this pull request as draft February 26, 2025 21:20
@azchohfi azchohfi force-pushed the alzollin/ortgenaiichatclient branch from d9f7f6a to 1a555d9 Compare February 26, 2025 21:27
@azchohfi
Copy link
Contributor Author

Waiting for new version of ortgenai that will have this.

@azchohfi azchohfi force-pushed the alzollin/ortgenaiichatclient branch 2 times, most recently from 0b2f898 to 0abda9d Compare February 27, 2025 18:17
@azchohfi azchohfi marked this pull request as ready for review February 27, 2025 18:18
@azchohfi azchohfi removed the blocked label Feb 27, 2025
@azchohfi
Copy link
Contributor Author

@nmetulev Not sure if we want to publish this now or wait until ORTGenAI is stable (non-rc).

@azchohfi
Copy link
Contributor Author

Build should pass, but arm64 is not working yet, since rc1 doesn't have arm64 binaries on Microsoft.ML.OnnxRuntimeGenAI.

@azchohfi
Copy link
Contributor Author

Fixed the arm64 issue.

@azchohfi azchohfi force-pushed the alzollin/ortgenaiichatclient branch from a2ec62b to 762aaf5 Compare February 28, 2025 00:50
@azchohfi
Copy link
Contributor Author

rc2 has arm64 assemblies, so this should build. Still, it is not working as we need the SK fix. We should merge this only after #292, which will have the SK update.
microsoft/semantic-kernel#10989

@azchohfi
Copy link
Contributor Author

@nmetulev we should be unblocked, but we should probably wait until stable 0.7.0 to merge this.

@azchohfi azchohfi removed the blocked label Mar 19, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants