-
Notifications
You must be signed in to change notification settings - Fork 1.9k
bug: Warn about Azure OpenAI completions file-incompatibility #4048
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.
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
base: main
Are you sure you want to change the base?
Changes from 3 commits
e37b0bc
91808ff
7cdf685
e6d4f06
9e0a102
5c45e72
cb6aa6b
17c6c25
63d7e6d
91b8337
a20068c
098f5ec
584a99d
61e4324
67c8371
caec194
ee4a419
b663951
aff00f3
22d356d
a10f9f1
f882fe8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🚩 Potential bypass of document support check when In (Refers to line 1245) Was this helpful? React with 👍 or 👎 to provide feedback.
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. IMO the for future reference, if we found these inconsistencies, having a single dataclass that uses property to validate combinations would be a better approach than checking multiple flags, since adding a check branch here would bloat this out further |
Uh oh!
There was an error while loading. Please reload this page.