Fix query parameter (#64)

* Fix query parameter

* update changelog

CHANGELOG.md

@@ -1,3 +1,13 @@
+### 0.9.1
+
+* Fixed a bug in `request_body` that prevented `query` argument from being passed to `HTTP.jl`.
+* Updated completion model in the unit tests suite (`ada` series has been deprecated).
+* Added warnings to `create_edit` that it's deprecated by OpenAI. Disabled tests for `get_usage_status` and `create_edit` functions, as they cannot be tested via API.
+
+### 0.9.0
+
+* Added OpenAI Assistants API
+
 ### 0.8.7
 
 * disable `test/usage.jl` in `test/runtests.jl` to close issue: https://github.com/JuliaML/OpenAI.jl/issues/46

Project.toml

@@ -1,7 +1,7 @@
 name = "OpenAI"
 uuid = "e9f21f70-7185-4079-aca2-91159181367c"
 authors = ["Rory Linehan @rory-linehan", "Marius Fersigan @algunion", "RexWzh @RexWzh", "Thatcher Chamberlin @ThatcherC", "Nicu Stiurca @nstiurca", "Peter @chengchingwen", "Stefan Wojcik @stefanjwojcik", "J S @svilupp", "Logan Kilpatrick @logankilpatrick", "Jerry Ling @Moelf"]
-version = "0.9.0"
+version = "0.9.1"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

src/OpenAI.jl

@@ -77,7 +77,6 @@ end
 
 function request_body(url, method; input, headers, query, kwargs...)
  input = isnothing(input) ? [] : input
- query = isnothing(query) ? [] : query
 
  resp = HTTP.request(method,
  url;
@@ -393,6 +392,8 @@ end
 """
 Create edit
 
+Note: This functionality is not accessible via API anymore -- see https://platform.openai.com/docs/deprecations
+
 # Arguments:
 - `api_key::String`: OpenAI API key
 - `model_id::String`: Model id (e.g. "text-davinci-edit-001")
@@ -410,6 +411,7 @@ function create_edit(api_key::String,
  instruction::String;
  http_kwargs::NamedTuple = NamedTuple(),
  kwargs...)
+ @warn "This functionality is not accessible via API anymore -- see https://platform.openai.com/docs/deprecations"
  return openai_request("edits",
  api_key;
  method = "POST",

test/completion.jl

@@ -4,21 +4,24 @@
  @test false
  end
  r = create_completion(ENV["OPENAI_API_KEY"],
- "text-ada-001";
+ "gpt-3.5-turbo-instruct";
  prompt = "Say \"this is a test\"")
  println(r.response["choices"][begin]["text"])
  if !=(r.status, 200)
  @test false
  end
 end
 
-@testset "create edit" begin
- r = create_edit(ENV["OPENAI_API_KEY"],
- "text-davinci-edit-001",
- "Fix this piece of text for grammatical errors",
- input = "I hav ben riting sence i wuz 5")
- println(r.response["choices"][begin]["text"])
- if !=(r.status, 200)
- @test false
+# This functionality is not accessible via API anymore -- see https://platform.openai.com/docs/deprecations
+@test_skip begin
+ @testset "create edit" begin
+ r = create_edit(ENV["OPENAI_API_KEY"],
+ "gpt-3.5-turbo-instruct",
+ "Fix this piece of text for grammatical errors",
+ input = "I hav ben riting sence i wuz 5")
+ println(r.response["choices"][begin]["text"])
+ if !=(r.status, 200)
+ @test false
+ end
  end
 end

test/usage.jl

@@ -1,14 +1,17 @@
 # Get usage status of an api
 
 # See https://github.com/JuliaML/OpenAI.jl/issues/46
-@testset "usage information" begin
- provider = OpenAI.OpenAIProvider(ENV["OPENAI_API_KEY"], "https://api.openai.com/v1", "")
- (; quota, usage, daily_costs) = get_usage_status(provider, numofdays = 5)
- @test quota > 0
- @test usage >= 0
- @test length(daily_costs) == 5
- println("Total quota: $quota")
- println("Total usage: $usage")
- costs = [sum(item["cost"] for item in day.line_items) for day in daily_costs]
- println("Recent costs(5 days): $costs")
-end
+# This functionality is not accessible via API anymore -- see https://platform.openai.com/docs/api-reference/administration
+@test_skip begin
+ @testset "usage information" begin
+ provider = OpenAI.OpenAIProvider(ENV["OPENAI_API_KEY"], "https://api.openai.com/v1", "")
+ (; quota, usage, daily_costs) = get_usage_status(provider, numofdays = 5)
+ @test quota > 0
+ @test usage >= 0
+ @test length(daily_costs) == 5
+ println("Total quota: $quota")
+ println("Total usage: $usage")
+ costs = [sum(item["cost"] for item in day.line_items) for day in daily_costs]
+ println("Recent costs(5 days): $costs")
+ end
+end