Skip to content

Commit 19d2444

Browse files
authored
Bugfix: Add param positional arg to InvalidRequestError (openai#573)
I was testing some code and I got this error: ``` File "/usr/local/lib/python3.10/site-packages/openai/api_resources/abstract/engine_api_resource.py", line 37, in class_url raise error.InvalidRequestError( TypeError: InvalidRequestError.__init__() missing 1 required positional argument: 'param' ``` So I checked this file and saw that in a few cases, InvalidRequestError was missing the second positional argument `param`
1 parent 532d3df commit 19d2444

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

openai/api_resources/abstract/engine_api_resource.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,13 @@ def class_url(
3535
if typed_api_type in (ApiType.AZURE, ApiType.AZURE_AD):
3636
if not api_version:
3737
raise error.InvalidRequestError(
38-
"An API version is required for the Azure API type."
38+
"An API version is required for the Azure API type.",
39+
"api_version"
3940
)
4041
if engine is None:
4142
raise error.InvalidRequestError(
42-
"You must provide the deployment name in the 'engine' parameter to access the Azure OpenAI service"
43+
"You must provide the deployment name in the 'engine' parameter to access the Azure OpenAI service",
44+
"engine"
4345
)
4446
extn = quote_plus(engine)
4547
return "/%s/%s/%s/%s?api-version=%s" % (
@@ -269,7 +271,8 @@ def instance_url(self):
269271
api_version = self.api_version or openai.api_version
270272
if not api_version:
271273
raise error.InvalidRequestError(
272-
"An API version is required for the Azure API type."
274+
"An API version is required for the Azure API type.",
275+
"api_version"
273276
)
274277
base = self.OBJECT_NAME.replace(".", "/")
275278
url = "/%s/%s/%s/%s/%s?api-version=%s" % (

0 commit comments

Comments
 (0)