Skip to content

Update ReadME / avoid chances for same attribute names in hijack / add h*w assertion#6

Open
aria1th wants to merge 3 commits intotfernd:mainfrom
aria1th:main
Open

Update ReadME / avoid chances for same attribute names in hijack / add h*w assertion#6
aria1th wants to merge 3 commits intotfernd:mainfrom
aria1th:main

Conversation

@aria1th
Copy link

@aria1th aria1th commented Nov 11, 2023

The cursed nature of hijack contains chances to make duplicate attribute names, especially ControlNet.

Some code snippets are changed for future references.

@aria1th
Copy link
Author

aria1th commented Nov 12, 2023

Note : Consider the following case.
8740 = 76 * 115

Whilas the aspect ratio is 1.5, the values are:

(8740*1.5)**0.5 =114.49890829173874
Thus the rounded value is 114.

This will fail the h*w = hw assertion.

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.

1 participant