Enhancements to Package Management, Documentation, Error Handling and implement fix argument #26
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I would like to add that there are still some type errors in the run openai and anthropic functions but since I couldn't test them myself I didn't try to fix those. |
…rove prompts to get better responses based on the os
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to the
wutproject, focusing on adding package metadata, updating dependencies, and improving the codebase. The most important changes include adding aPKGBUILDfile, removing thePipfile, updating theREADME.md, and modifying several functions inwut/utils.pyfor enhanced functionality and error handling.Package Metadata and Dependencies:
PKGBUILD: Added a newPKGBUILDfile to define the package metadata and build process for thewut-cli-gitproject when using an arch linux based system.Pipfile: Removed thePipfileto transition from Pipenv to a standard requirements.txt file.Documentation:
README.md: Added instructions for customizing the Ollama endpoint and model in the environment variables.Code Improvements:
wut/utils.py: Updated therun_ollamato handle the optional use of a remote endpointmypy.ini: Added a newmypy.iniconfiguration file to ignore missing library stubs during type checking.Error Handling and Enhancements:
wut/utils.py: Enhanced therun_ollamafunction to handle connection errors and response errors more gracefully, and updated therun_openaiandrun_anthropicfunctions to ensure they return non-empty strings. [1] [2] [3]This pull request includes several changes to the
wutCLI tool, focusing on package management, environment customization, and code improvements. The most important changes include adding a newPKGBUILDfile, updating theREADME.mdfor Ollama configuration, and various code improvements inwut/utils.py.Package Management:
PKGBUILDfile to support building the package from source, including necessary metadata and build instructions.Pipfileto transition to a different package management approach.Documentation:
README.mdto include instructions for customizing the Ollama endpoint and model through environment variables.Code Improvements:
mypy.iniconfiguration to ignore missing imports, improving type-checking setup.wut/utils.pyto improve code readability and robustness, including changes to handle empty strings and better error handling for Ollama integration. [1] [2] [3] [4] [5] [6] [7] [8] [9]Error Handling:
wut/utils.py, including specific messages for connection errors and missing models.