Skip to content

fix: fix syntax errors#15179

Merged
AUTOMATIC1111 merged 2 commits into
AUTOMATIC1111:devfrom
llnancy:master
Mar 16, 2024
Merged

fix: fix syntax errors#15179
AUTOMATIC1111 merged 2 commits into
AUTOMATIC1111:devfrom
llnancy:master

Conversation

@llnancy

@llnancy llnancy commented Mar 8, 2024

Copy link
Copy Markdown
Contributor

Description

python syntax error.

Screenshots/videos:

image

Checklist:

@llnancy llnancy requested a review from AUTOMATIC1111 as a code owner March 8, 2024 09:27
@llnancy llnancy changed the base branch from master to dev March 8, 2024 09:29
@AUTOMATIC1111

AUTOMATIC1111 commented Mar 8, 2024

Copy link
Copy Markdown
Owner

Clearly not a syntax error because the program obviously runs... A proper fix would be scalers: list.

@light-and-ray

Copy link
Copy Markdown
Contributor

Hm, even linter doesn't detect this. The code is not dead. Maybe linter and interpretator just ignores everything after :, it's funny XD

@AUTOMATIC1111

Copy link
Copy Markdown
Owner

= [] is a meaningful change and since it works without it, it's not needed. This should be just scalers: list

@AUTOMATIC1111 AUTOMATIC1111 merged commit cb09e1e into AUTOMATIC1111:dev Mar 16, 2024
AUTOMATIC1111 added a commit that referenced this pull request Mar 16, 2024
ruchej pushed a commit to ruchej/stable-diffusion-webui that referenced this pull request Sep 30, 2024
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.

3 participants