stable-diffusion.cpp icon indicating copy to clipboard operation
stable-diffusion.cpp copied to clipboard

feat: support stream cmd and lowvram options

Open Cyberhan123 opened this issue 1 year ago • 6 comments

This is the draft. Implementing this PR means that we can have greater expansion capabilities in the future, such as easier implementation of CLIP interrogators. At present, this PR has not been completed. I still need to test and complete the logic of the cli. Since the changes are relatively large, I thought I would release them for everyone to see. cc @leejet @Green-Sky @FSSRepo

Cyberhan123 avatar Jan 23 '24 11:01 Cyberhan123

There are two IndentCaseLabels in the current .clang-format. I try to set them to true or false, but they will format the code that I have not modified. :flushed:

Cyberhan123 avatar Jan 23 '24 11:01 Cyberhan123

Its hard to review with 90% formatting changes.

Green-Sky avatar Jan 23 '24 13:01 Green-Sky

Its hard to review with 90% formatting changes.

I don’t know much about the formatting method of C++. I used the script in the project to restore it, but with little success. I plan to adjust it manually tomorrow.

Cyberhan123 avatar Jan 23 '24 16:01 Cyberhan123

Thank you for your comment, I will adjust these issues. When I was writing, I found that manually adjusting the code format while writing was a disaster. I plan to do it when This PR is completed.

Cyberhan123 avatar Jan 24 '24 14:01 Cyberhan123

I removed the first IndentCaseLabels in the latest master branch, which should be a legacy configuration. I recommend that you use clang-format's command line tool to format your code for consistency.

leejet avatar Jan 29 '24 15:01 leejet

I removed the first IndentCaseLabels in the latest master branch, which should be a legacy configuration. I recommend that you use clang-format's command line tool to format your code for consistency.

Thank you very much, I don't have time now.

Cyberhan123 avatar Jan 31 '24 02:01 Cyberhan123