Develop #34
Merged
Develop #34
Conversation
This file contains 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
Feat/minor updates
Retrieve API key from OPENAI_API_KEY in case SARATHI_OPENAI_API_KEY is undefined for backward compatibility
* `retrieve_api_key()` function to retrieve API key from environment variable SARATHI_OPENAI_API_KEY or OPENAI_API_KEY (#54) * Added new functions `retrieve_llm_url()`, `retrieve_model_name()`, and `call_llm_model()` to retrieve the OpenAI API endpoint URL, L
…ling, and clean up code. Add default values for missing variables. Update print statements. Remove unnecessary prints. Improve readability and maintainability.
- retrieve_model_name function. This change ensures the use of a - more advanced model when the environment variable is not set.
Feat/allow local urls
- `call_llm_model` function. If the model is not found in - `prompt_info`, it falls back to using a default model - retrieved from `retrieve_model_name()`. This improves - robustness against missing model information.
- retrieval, enhancing flexibility. - Adjustments ensure that the model can be fetched directly from - the prompt info dictionary if available, otherwise falling back - to environment variables.
…r the - CLI coding assistant, including support for local LLM model endpoints. - Refactored `call_llm.py` to improve model name retrieval and handling.
…Isort. - Modify `retrieve_model_name` function to use the model from `prompt_info` - as default if `OPENAI_MODEL_NAME` is not set, ensuring better model retrieval.
src/sarathi/cli/sgit.py
Outdated
| @@ -71,6 +71,7 @@ def execute_cmd(args): | |||
| elif args.git_sub_cmd == "autocommit": | |||
| generated_commit_msg = generate_commit_message() | |||
| if generated_commit_msg: | |||
| print("**Below is the generated commit messaged **\n") | |||
There was a problem hiding this comment.
@abhishek9sharma I think here messaged should be message.
- Updated message from "messaged" to "message" for clarity and correctness.
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.
No description provided.