Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added multi-model selection and bulk sub_* function rename features and merged with Vulchat #19

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

warsang
Copy link

@warsang warsang commented Apr 14, 2023

Hi!

I forked this intially for personal use but decided I would share some of the changes I made (I mostly wanted model selection and being able to bulk rename functions).
Beware, a few edits may be needed as I couldn't 100% test everything as I ended up running out of tokens but the bulk of the code works and I fixed whatever was broken while I still had tokens.

Features include:

  • Model selection (I use a tkinter pop-up; there may be an ida-only api way to do it but this was just easy to implement)
  • Merge with Vulchat ( https://github.com/ke0z/VulChatGPT ): Vulchat is still using dali-003 . There will probably be better perf/results when leveraging these new models and gpt 3.5/4.
    I couldn't test the two below because I ran out of token so a few edits may be needed to get these to work 100%
  • Rename function: Similar to rename_variables but will rename the function
  • Rename all sub_: This will rename all sub_ functions that are less than 800 characters long (we can probably bump this up but I needed threshold). It has the potential to make a lot of api calls so a warning before using it may be desirable.

- Added support to read from a config file
…odels available)

- Changed readme.md and fixed bugs when reading config
…odels available)

- Changed readme.md and fixed bugs when reading config
- Added the rename all sub_* action
- Removed the entry about the config file as managing an external config file is annoying.
@JusticeRage
Copy link
Owner

Wow, thanks a lot for your PR! I'll look into the code and merge as soon as I get a chance!

JusticeRage added a commit that referenced this pull request Apr 27, 2023
Added support for GPT-4 (closes #18).
Added UI elements to switch between models at runtime.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants