Skip to content
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

UI / worker image mismatch with settings #15

Open
robin-raymond opened this issue Nov 11, 2023 · 1 comment
Open

UI / worker image mismatch with settings #15

robin-raymond opened this issue Nov 11, 2023 · 1 comment

Comments

@robin-raymond
Copy link
Contributor

When I'd switch from multi to hires I'd see these asserts and 'array length mismatch' happen, so I dug into the code. The issue is that the canvas and the UI get out of sync and when the target system changes the image in the dithertron is the old image while the settings is set to the new setting. This causes the logic that grabs the image width/height to miscalculate because it's not expecting to see a 1:1 matching for the image.

Can the code be fixed that the dithertron is reset, and the image is guaranteed to always be reinitialized prior to redoing the settings? I've not wanted to touch the UI stuff for fear I could really mess that area up.

@robin-raymond
Copy link
Contributor Author

The switch can also cause assertion failures, which look concerning but only happen because the wrong image is being fed into the canvas during the switching process. They aren't harmful per say though because the correct image is fed in eventually but the asserts look bad in the output.

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

No branches or pull requests

1 participant