Merge pull request #1684 from swirlai/ds-4629-develop

cherry-pick from main, removing temperature parameter for ChatGPT-5 …

Author: erikspears

swirl/connectors/gen_ai.py

@@ -79,8 +79,7 @@ def execute_search(self, session=None):
             messages=[
                 {"role": "system", "content": self.system_guide},
                 {"role": "user", "content": self.query_to_provider},
-            ],
-            temperature=0
+            ]
         )
         message = completions.choices[0].message.content
         self.found = 1

swirl/processors/gen_ai_query.py

@@ -106,16 +106,14 @@ def process(self, client=None):
                 messages=[
                     {"role": "system", "content": self.system_guide},
                     {"role": "user", "content": self.prompt.format(query_string=self.query_string)    },
-                ],
-                temperature=0
+                ]
             )
             response = client.openai_client.chat.completions.create(
                 model=client.get_model(),
                 messages=[
                     {"role": "system", "content": self.system_guide},
                     {"role": "user", "content": self.prompt.format(query_string=self.query_string)    },
-                ],
-                temperature=0
+                ]
             )
             message = response.choices[0].message.content
             logger.info(f"ChatGPT Response: {message}")

swirl/processors/rag.py

@@ -232,8 +232,7 @@ def background_process(self):
                 messages=[
                     {"role": "system", "content": rag_prompt.get_role_system_guide_text()},
                     {"role": "user", "content": new_prompt_text},
-                ],
-                temperature=0
+                ]
             )
             model_response = completions_new.choices[0].message.content
             logger.warning(f'RAG: fetch_prompt_errors follow:')

swirl/tests/tests.py

@@ -601,8 +601,7 @@ def test_cgptqp_1():
             messages=[
                 {"role": "system", "content": "You are helping a user formulate better queries"},
                 {"role": "user", "content": "Write a more precise query of similar length to this : gig economy"}
-            ],
-            temperature=0
+            ]
         )
 
 @pytest.mark.django_db
@@ -630,8 +629,8 @@ def test_cgptqp_2():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "Write a more precise query of similar length to this : gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 @pytest.mark.django_db
 def test_cgptqp_3():
@@ -658,8 +657,8 @@ def test_cgptqp_3():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "Write a more precise query of similar length to this : gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 @pytest.mark.django_db
 def test_cgptqp_4():
@@ -688,8 +687,8 @@ def test_cgptqp_4():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "This should be used: gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 
 @pytest.mark.django_db
@@ -721,8 +720,8 @@ def test_cgptqp_5():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "This should be used: gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 
 @pytest.mark.django_db
@@ -754,8 +753,8 @@ def test_cgptqp_6():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "This should be used: gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 
 @pytest.mark.django_db
@@ -787,8 +786,8 @@ def test_cgptqp_7():
         mock_create.assert_called_once_with(model=TEST_AI_MODEL, messages=[
                 {"role": "system", "content": "You are a malevolent dictator"},
                 {"role": "user", "content":   "This should be used: gig economy"}
-            ],
-            temperature=0)
+            ]
+        )
 
 
 @pytest.fixture