Skip to content

Commit

Permalink
updated note
Browse files Browse the repository at this point in the history
  • Loading branch information
eavanvalkenburg committed Jan 9, 2025
1 parent e7fc62d commit f83002c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions python/semantic_kernel/connectors/ai/realtime_client_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,22 +17,22 @@
####
# TODO (eavanvalkenburg): Move to ADR
# Receiving:
# Option 1: Events and Contents split (current)
# Option 1: Events and Contents split
# - content received through main receive_content method
# - events received through event callback handlers
# Option 2: Everything is Content
# - content (events as new Content Type) received through main receive_content method
# Option 3: Everything is Event
# Option 3: Everything is Event (current)
# - receive_content method is removed
# - events received through main listen method
# - default event handlers added for things like errors and function calling
# - built-in vs custom event handling - separate or not?
# Sending:
# Option 1: Events and Contents split (current)
# Option 1: Events and Contents split
# - send_content and send_event
# Option 2: Everything is Content
# - single method needed, with EventContent type support
# Option 3: Everything is Event
# Option 3: Everything is Event (current)
# - send_event method only, Content is part of event data
####

Expand Down

0 comments on commit f83002c

Please sign in to comment.