Skip to content

avoid 'prefill' border when 'inpaint_border' is disabled - #231

Merged
kostarion merged 2 commits into
anima_stablefrom
cut_pow/prefill_fix
May 12, 2023
Merged

kostarion merged 2 commits into
anima_stablefrom
cut_pow/prefill_fix

Conversation

@kostarion

Copy link
Copy Markdown
Contributor

No description provided.

@kostarion
kostarion requested a review from pharmapsychotic May 12, 2023 16:41
Comment thread src/stability_sdk/animation.py Outdated
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Comment thread src/stability_sdk/animation.py Outdated
if args.border == 'wrap' and args.animation_mode != '2D':
args.border = 'reflect'
logger.warning(f"Border 'wrap' is only supported in 2D mode, switching to '{args.border}'.")
if args.border == 'prefill' and args.animation_mode in ('2D', '3D warp') and args.inpaint_border == False:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be args.animation_mode != '3D render' instead of checking against 2D and 3D warp? Otherwise will hit same issue with Video Input potentially.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Video Input mode doesn't apply any bordering, so the selected border is idle in this case

@kostarion
kostarion merged commit 3a06ade into anima_stable May 12, 2023
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.

2 participants