Skip to content

Dev#164

Merged
madina1203 merged 2 commits into
mainfrom
dev
Sep 8, 2025
Merged

Dev#164
madina1203 merged 2 commits into
mainfrom
dev

Conversation

@madina1203
Copy link
Copy Markdown
Collaborator

@madina1203 madina1203 commented Sep 8, 2025

PR Type

configuration changes


Description

  • Add new configuration section for llm_gpt_5

  • Update configuration parameters in params.ini


Diagram Walkthrough

flowchart LR
  A["Existing Configuration"] -- "Add new section" --> B["Configuration for llm_gpt_5"]
Loading

File Walkthrough

Relevant files
Configuration changes
params.ini
Add configuration for `llm_gpt_5` model                                   

app/config/params.ini

  • Added new section [llm_gpt_5]
  • Set id, temperature, and max_retries for gpt-5
+5/-0     


Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 8, 2025
    @coderabbitai
    Copy link
    Copy Markdown

    coderabbitai Bot commented Sep 8, 2025

    Caution

    Review failed

    An error occurred during the review process. Please try again later.

    ✨ Finishing Touches
    🧪 Generate unit tests
    • Create PR with unit tests
    • Post copyable unit tests in a comment
    • Commit unit tests in branch dev

    Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

    ❤️ Share

    Comment @coderabbitai help to get the list of available commands and usage tips.

    @qodo-code-review
    Copy link
    Copy Markdown

    qodo-code-review Bot commented Sep 8, 2025

    PR Reviewer Guide 🔍

    (Review updated until commit daef213)

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Config Consistency

    Please verify that the new configuration section for llm_gpt_5 adheres to the existing parameter conventions and that the provided values are consistent with system expectations.

    [llm_gpt_5]
    id = gpt-5
    temperature = 1
    max_retries = 3
    

    @github-actions
    Copy link
    Copy Markdown
    Contributor

    github-actions Bot commented Sep 8, 2025

    Persistent review updated to latest commit daef213

    @github-actions
    Copy link
    Copy Markdown
    Contributor

    github-actions Bot commented Sep 8, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @dosubot dosubot Bot added Configuration changes documentation Improvements or additions to documentation labels Sep 8, 2025
    @qodo-code-review
    Copy link
    Copy Markdown

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link
    Copy Markdown

    @coderabbitai coderabbitai Bot left a comment

    Choose a reason for hiding this comment

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

    Actionable comments posted: 1

    🧹 Nitpick comments (1)
    docs/index.md (1)

    21-21: Nit: standardize trailing slash with Quickstart

    Quickstart uses a trailing slash; this one doesn’t. If the site canonicalizes to one form, mirror it here for consistency and cleaner link checking.

    Apply if you choose trailing slashes:

    -👉 **Try the MetaboT Web App**: [https://metabot.holobiomicslab.eu](https://metabot.holobiomicslab.eu)
    +👉 **Try the MetaboT Web App**: [https://metabot.holobiomicslab.eu/](https://metabot.holobiomicslab.eu/)
    📜 Review details

    Configuration used: CodeRabbit UI

    Review profile: CHILL

    Plan: Pro

    📥 Commits

    Reviewing files that changed from the base of the PR and between c2e0761 and daef213.

    ⛔ Files ignored due to path filters (1)
    • app/config/params.ini is excluded by !**/*.ini
    📒 Files selected for processing (2)
    • docs/getting-started/quickstart.md (1 hunks)
    • docs/index.md (1 hunks)
    🔇 Additional comments (2)
    docs/getting-started/quickstart.md (1)

    5-5: URL update LGTM

    Switch to holobiomicslab.eu looks correct.

    docs/index.md (1)

    21-21: URL update LGTM

    Good swap to holobiomicslab.eu.

    Welcome to the Quick Start Guide for 🧪 MetaboT 🍵. This guide will help you quickly run and test the application.

    👉 **Try the MetaboT Web App Demo**: [https://metabot.holobiomicslab.cnrs.fr](https://metabot.holobiomicslab.cnrs.fr) — no installation needed!
    👉 **Try the MetaboT Web App Demo**: [https://metabot.holobiomicslab.eu/](https://metabot.holobiomicslab.eu/) — no installation needed!
    Copy link
    Copy Markdown

    Choose a reason for hiding this comment

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

    💡 Verification agent

    🧩 Analysis chain

    Consistency: align trailing slash across docs

    This link includes a trailing slash, while docs/index.md (Line 21) does not. Pick one style to avoid mixed canonicals.

    Run to confirm all old domains are gone and the new URL resolves:


    🏁 Script executed:

    #!/bin/bash
    set -euo pipefail
    
    echo "Searching for any lingering cnrs.fr references..."
    rg -nI --hidden --glob '!.git' 'holobiomicslab\.cnrs\.fr' || echo "No cnrs.fr references found."
    
    echo "Checking new endpoint reachability and canonicalization..."
    for url in "https://metabot.holobiomicslab.eu" "https://metabot.holobiomicslab.eu/"; do
      echo -e "\nHEAD $url"
      curl -sI "$url" | sed -n '1p;/^location:/Ip'
    done

    Length of output: 392


    🏁 Script executed:

    #!/bin/bash
    set -euo pipefail
    
    echo "Searching for lingering cnrs.fr references..."
    rg -n 'holobiomicslab\.cnrs\.fr' || echo "No cnrs.fr references found."
    
    echo "Checking all occurrences of metabot.holobiomicslab.eu in docs/index.md"
    rg -n -C1 'metabot\.holobiomicslab\.eu' docs/index.md

    Length of output: 997


    Fix legacy domain in README and standardize trailing slash in Quickstart

    • In docs/getting-started/quickstart.md (Line 5), remove the trailing slash so the link matches docs/index.md:
      change
      [https://metabot.holobiomicslab.eu/](https://metabot.holobiomicslab.eu/)
      to
      [https://metabot.holobiomicslab.eu](https://metabot.holobiomicslab.eu)
    • In README.md (Line 16), replace the old “holobiomicslab.cnrs.fr” URL with “metabot.holobiomicslab.eu” to remove the legacy domain reference.
    🤖 Prompt for AI Agents
    In docs/getting-started/quickstart.md around line 5 and README.md around line
    16, update the demo links to the canonical metabot domain: in quickstart.md
    remove the trailing slash so the link becomes
    [https://metabot.holobiomicslab.eu](https://metabot.holobiomicslab.eu) (no
    trailing slash) to match docs/index.md, and in README.md replace any occurrence
    of the legacy holobiomicslab.cnrs.fr URL with https://metabot.holobiomicslab.eu
    to remove the legacy domain reference.
    

    @madina1203 madina1203 merged commit db4592c into main Sep 8, 2025
    2 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    Configuration changes documentation Improvements or additions to documentation Review effort 1/5 size:XS This PR changes 0-9 lines, ignoring generated files.

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    1 participant