Skip to content

Conversation

joneavila
Copy link
Contributor

As of the latest version of macOS (15.0.1), the --force-upcast-attention flag is required to fix a "black image" bug (#3521). Similar to #4548, this PR checks whether the macOS version is affected and automatically forces upcast attention.

@comfyanonymous comfyanonymous added the Run-CI-Test This is an administrative label to tell the CI to run full automatic testing on this PR now. label Oct 10, 2024
Copy link

(Automated Bot Message) CI Tests are running, you can view the results at https://ci.comfy.org/?branch=5191%2Fmerge

@comfyanonymous comfyanonymous merged commit 4b2f0d9 into comfyanonymous:master Oct 10, 2024
16 of 18 checks passed
@davidbenton
Copy link
Contributor

I'm glad to see this updated for MacOS 15, but it's unfortunately already out of date. I'm running 15.1 and getting black output. Would it make sense to assume this behavior until next major version (i.e. (14, 5) <= macos_version < (16, 0))? Or maybe just make it the default going forward ((14, 5) <= macos_version )? I'm not sure what the risks of the second option would be.

@comfyanonymous
Copy link
Owner

I assume they are going to fix it at some point so I'll just keep bumping the version up until they do.

@davidbenton
Copy link
Contributor

🫡 Thank you for your service

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Run-CI-Test This is an administrative label to tell the CI to run full automatic testing on this PR now.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants