Skip to content

docs#1552

Merged
mrT23 merged 1 commit intomainfrom
tr/custom_reasoning_model
Feb 18, 2025
Merged

docs#1552
mrT23 merged 1 commit intomainfrom
tr/custom_reasoning_model

Conversation

@mrT23
Copy link
Copy Markdown
Contributor

@mrT23 mrT23 commented Feb 18, 2025

PR Type

Documentation


Description

  • Commented out a section about deepseek-r1 configuration.

  • Updated usage guide for Qodo Merge models.


Changes walkthrough 📝

Relevant files
Documentation
qodo_merge_models.md
Commented out `deepseek-r1` configuration section               

docs/docs/usage-guide/qodo_merge_models.md

  • Commented out the section for deepseek-r1 configuration.
  • Adjusted documentation to clarify model restrictions.
  • +5/-5     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @qodo-free-for-open-source-projects
    Copy link
    Copy Markdown
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-free-for-open-source-projects
    Copy link
    Copy Markdown
    Contributor

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    Learned
    best practice
    Use explicit notices for unavailable features instead of hiding them with HTML comments to improve documentation clarity and user experience

    Instead of using HTML-style comments to hide documentation content, consider
    using a more explicit approach to indicate deprecated or unavailable features.
    Add a note explaining why the deepseek-r1 configuration is no longer available
    or when it will be available again.

    docs/docs/usage-guide/qodo_merge_models.md [27-31]

    -[//]: # (To restrict Qodo Merge to using only `deepseek-r1` us-hosted, add this setting:)
    -[//]: # (```)
    -[//]: # ([config])
    -[//]: # (model="deepseek/r1")
    -[//]: # (```)
    +> **Note:** The `deepseek-r1` model configuration is temporarily unavailable.
    +> This feature will be re-enabled in a future release.
     
    +<!-- Original configuration for reference:
    +```
    +[config]
    +model="deepseek/r1"
    +```
    +-->
    +
    • Apply this suggestion
    Suggestion importance[1-10]: 6
    Low
    • More
    • Author self-review: I have reviewed the PR code suggestions, and addressed the relevant ones.

    @mrT23 mrT23 merged commit 6efcd61 into main Feb 18, 2025
    2 checks passed
    @mrT23 mrT23 deleted the tr/custom_reasoning_model branch February 18, 2025 15:42
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    1 participant