Skip to content

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

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

avoid 'prefill' border when 'inpaint_border' is disabled#231
kostarion merged 2 commits intoanima_stablefrom
cut_pow/prefill_fix

Conversation

@kostarion
Copy link
Contributor

No description provided.

@kostarion kostarion requested a review from pharmapsychotic May 12, 2023 16:41
@sonarqubecloud
Copy link

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

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
Member

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
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