Merge pull request #199 from psu-libraries/bugfix-makes-default-for-llm-model

jlandiseigsti · web-flow · commit 7b358b272fe7 · 2025-11-03T14:26:14.000-07:00
"Default" is new default
diff --git a/app/jobs/image_alt_text_job.rb b/app/jobs/image_alt_text_job.rb
@@ -13,7 +13,7 @@ def perform(job_uuid, tmp_path)
     alt_text = client.process_image(
       tmp_path,
       prompt: Rails.root.join('prompt.txt').read,
-      model_id: ENV.fetch('LLM_MODEL', 'nil')
+      model_id: ENV.fetch('LLM_MODEL', 'default')
     )
     job.update(
       status: 'completed',
diff --git a/spec/jobs/image_alt_text_job_spec.rb b/spec/jobs/image_alt_text_job_spec.rb
@@ -27,7 +27,7 @@
 
       it 'calls the Alt Text gem' do
         expect(alt_text_gem).to have_received(:process_image).with(
-          /.+\.jpg/, prompt: File.read('prompt.txt'), model_id: ENV.fetch('LLM_MODEL', nil)
+          /.+\.jpg/, prompt: File.read('prompt.txt'), model_id: ENV.fetch('LLM_MODEL', 'default')
         )
       end
 

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ def perform(job_uuid, tmp_path)`
`13`	`13`	`alt_text = client.process_image(`
`14`	`14`	`tmp_path,`
`15`	`15`	`prompt: Rails.root.join('prompt.txt').read,`
`16`		`- model_id: ENV.fetch('LLM_MODEL', 'nil')`
	`16`	`+ model_id: ENV.fetch('LLM_MODEL', 'default')`
`17`	`17`	`)`
`18`	`18`	`job.update(`
`19`	`19`	`status: 'completed',`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@`
`27`	`27`
`28`	`28`	`it 'calls the Alt Text gem' do`
`29`	`29`	`expect(alt_text_gem).to have_received(:process_image).with(`
`30`		`- /.+\.jpg/, prompt: File.read('prompt.txt'), model_id: ENV.fetch('LLM_MODEL', nil)`
	`30`	`+ /.+\.jpg/, prompt: File.read('prompt.txt'), model_id: ENV.fetch('LLM_MODEL', 'default')`
`31`	`31`	`)`
`32`	`32`	`end`
`33`	`33`