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

AI assisted Script Editiing: removed dependencies, refactored OpenAI code, added anthropic Claude #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

haesleinhuepf
Copy link
Member

Hi Curtis @ctrueden ,

I was working on LLM integration in the Script Editor again. The library we used before was archived by its developer and had a lot of dependencies, that's why I removed it. I'm now accessing the LLMs directly via their REST API. Thus, we have now minimal dependencies, no additional jar files that need to be shipped via the update site.

Furthermore, I used the opportunity to refactor code, and add support for Anthropic's claude. Support for Gemini, Ollama etc seems trivial, and could be added via a follow-up PR. I also renamed "OpenAI Options" to "LLM Service Options" and also optimized the default prompt a bit.

Let me know what you think!

Best,
Robert

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/ai-assisted-code-generation-in-fijis-script-editor/100877/1

@haesleinhuepf haesleinhuepf changed the title removed dependencies, refactored OpenAI code, addend anthropic Claude removed dependencies, refactored OpenAI code, added anthropic Claude Aug 24, 2024
@haesleinhuepf haesleinhuepf changed the title removed dependencies, refactored OpenAI code, added anthropic Claude AI assisted Script Editiing: removed dependencies, refactored OpenAI code, added anthropic Claude Aug 27, 2024
@haesleinhuepf haesleinhuepf requested a review from ctrueden August 27, 2024 10:50
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.

2 participants