Skip to content

Conversation

@kuivi
Copy link
Collaborator

@kuivi kuivi commented Jan 13, 2026

  1. Config Restructuring (config.yml)

    Replaced flat model configuration with structured per-component settings: Before:

    model_type: "openai"
    model_name: "gpt-4.1-nano"
    model_name_rag: "gpt-4.1-nano"
    model_name_tools: "gpt-4.1-nano"
    model_name_combine_agent: "gpt-5-mini"
    model_name_agents: "gpt-4.1-nano"
    After:

    llm_rag:
    model_type: "openai"
    model_name: "gpt-4.1-nano"
    llm_smart:
    model_type: "openai"
    model_name: "gpt-4.1-nano"
    llm_combine:
    model_type: "openai"
    model_name: "gpt-4.1-nano"

  2. Code Updates (across 5 files)

    Updated all references to use new nested structure:
    config['model_name'] → config['llm_combine']['model_name']
    config['model_type'] → config['llm_combine']['model_type']
    config['model_name_rag'] → config['llm_rag']['model_name']
    config['model_name_tools'] → config['llm_smart']['model_name']
    Files modified:
    climsight_engine.py (47 lines changed)
    smart_agent.py (20 lines changed)
    streamlit_interface.py (12 lines changed)
    rag.py (2 lines changed)
    evaluation.py (2 lines changed)
    climsight.py (2 lines changed)

@kuivi kuivi requested review from dmpantiu and koldunovn January 13, 2026 17:17
Copy link
Collaborator

@dmpantiu dmpantiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request focuses on refactoring how Large Language Models (LLMs) are organized and configured within the climsight codebase. The primary goal is to move from a flat configuration structure to a more modular, "direct case" approach that splits configurations for different tasks (RAG, Smart Agent, and Synthesis).

@dmpantiu dmpantiu merged commit 1041cad into main Jan 15, 2026
4 checks passed
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