-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bug] [gui] Fix a bug of drawing mesh instacing that cpu/cuda objects have an offset when copying to vulkan object #6028
Conversation
This reverts commit b443eb0.
…ble objects) directly
✅ Deploy Preview for docsite-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
/rebase |
Rebase failed, please resolve the conflicts manually. |
May I merge this directly and solve the conflicts? It seems that rebase is a bit troublesome. |
@Morcki you can resolve merge conflict locally and the force push into this branch. |
… backends when copying to vulkan backend [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Cpu backend is with LLVM and add annotation for memcpy_cpu_to_vulkan [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci fix a bug when transforms is None, instance count must be 1 [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci fix a small bug change ssbo to device-local memory [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci
8bea7bb
to
8d5fdca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This only happend on Windows.