Skip to content

refactor: Update model_sdv2.yaml - #35

Merged
Haruko386 merged 1 commit into
Haruko386:mainfrom
Dimon0000000:main
Jul 29, 2026
Merged

refactor: Update model_sdv2.yaml#35
Haruko386 merged 1 commit into
Haruko386:mainfrom
Dimon0000000:main

Conversation

@Dimon0000000

@Dimon0000000 Dimon0000000 commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Update model_sdv2.yaml

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor
  • CI / build change
  • Other

Summary by CodeRabbit

  • Improvements
    • Updated image generation to use the Stable Diffusion 2.1 model, providing access to the latest supported model version while preserving existing image scaling behavior.

@Dimon0000000 Dimon0000000 added this to the ApDepth V2-1 milestone Jul 29, 2026
@Dimon0000000 Dimon0000000 added 🐞 bug Something isn't working ✨ enhancement New feature for better experience labels Jul 29, 2026
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 556cc4c7-7401-4596-a8e1-4b113219688d

📥 Commits

Reviewing files that changed from the base of the PR and between a57e06c and b627561.

📒 Files selected for processing (1)
  • config/model_sdv2.yaml

📝 Walkthrough

Walkthrough

The SDv2 model configuration changes the pretrained model identifier from stable-diffusion-2 to stable-diffusion-2-1, while leaving the latent scale factor unchanged.

Changes

SDv2 model configuration

Layer / File(s) Summary
Update pretrained model reference
config/model_sdv2.yaml
apdepth_pipeline now uses stable-diffusion-2-1 as its pretrained_path; latent_scale_factor remains 0.18215.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately reflects the main change to model_sdv2.yaml.
Description check ✅ Passed The description covers the summary and change type, but it is missing the Related Issue and Test Results sections from the template.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@Haruko386
Haruko386 merged commit 927b8f6 into Haruko386:main Jul 29, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in @ApDepth V2-1 Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working ✨ enhancement New feature for better experience

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants