GravityPhone / hatz

0 stars 0 forks source link

sweep: address error #14

Closed GravityPhone closed 3 months ago

GravityPhone commented 3 months ago

python .\main_controller.py Traceback (most recent call last): File "C:\smarthat\main_controller.py", line 6, in from assistant_manager import AssistantManager File "C:\smarthat\assistant_manager.py", line 2, in from openai import AssistantEventHandler ImportError: cannot import name 'AssistantEventHandler' from 'openai' (C:\Users\gravi.pyenv\pyenv-win\versions\3.9.0\lib\site-packages\openaiinit.py)

Administrator in smarthat on  main [!+] via 🐍 v3.9.0

handle this, we are refactoring for the change to streaming the response from the assistant once we make the thread btw

we still transcribe as normal and make the thread as normal but it's how we run that thread that we are interested in.

Checklist - [X] Modify `assistant_manager.py` ✓ https://github.com/GravityPhone/hatz/commit/c7da443ffdcab4f5f77326ac7f4dabd1bf5b687d [Edit](https://github.com/GravityPhone/hatz/edit/sweep/address_error_830b2/assistant_manager.py) - [X] Running GitHub Actions for `assistant_manager.py` ✓ [Edit](https://github.com/GravityPhone/hatz/edit/sweep/address_error_830b2/assistant_manager.py)
sweep-ai[bot] commented 3 months ago

🚀 Here's the PR! #16

See Sweep's progress at the progress dashboard!
💎 Sweep Pro: I'm using GPT-4. You have unlimited GPT-4 tickets. (tracking ID: d2ef19e6d1)
Install Sweep Configs: Pull Request

[!TIP] I can email you next time I complete a pull request if you set up your email here!


Actions (click)


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I think are relevant in decreasing order of relevance (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/GravityPhone/hatz/blob/871c46cac31077a88869950a3459716c963f476e/main_controller.py#L1-L14 https://github.com/GravityPhone/hatz/blob/871c46cac31077a88869950a3459716c963f476e/assistant_manager.py#L1-L28

Step 2: ⌨️ Coding

--- 
+++ 
@@ -1,9 +1,9 @@
 import openai
-from openai import AssistantEventHandler
+from openai.api import Assistant
 from typing_extensions import override
 import time

-class EventHandler(AssistantEventHandler):    
+class EventHandler(Assistant):    
   @override
   def on_text_created(self, text) -> None:
     print(f"\nassistant > ", end="", flush=True)

Ran GitHub Actions for c7da443ffdcab4f5f77326ac7f4dabd1bf5b687d:


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/address_error_830b2.


🎉 Latest improvements to Sweep:
  • New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
  • Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
  • Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description. Something wrong? Let us know.

This is an automated message generated by Sweep AI.