-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix details mask on low opencl memory #16003
Conversation
Due to the design of the details threshold mask we precalculate the scharr mask in demosaic module but we do **not** support doing that in tiling mode. In darktable-org#15999 and related reports in pixls and on github we ran into problems because of tiling the generated scharr mask did not cover the full roi of demosaic but just a part of it, this lead to crashes. Due to the way we commit parameters while preparing the pipeline we have to make sure about no tiling while committing params and also make sure in process and the cl variant.
When I tested the issue, I noticed that it also happens on CPU path. I had to use the notebook resources setting and disable opencl. |
I just tested it and it works in both CPU and openCL path. Thanks. |
Right. |
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.
Thanks!
Merged in master and 4.6.x branch. |
Release note would be: Fix crashes related to details mask if running on low OpenCL or system memory |
Due to the design of the details threshold mask we precalculate the scharr mask in demosaic module but we do not support doing that in tiling mode.
In #15999 and related reports in pixls and on github we ran into problems because of tiling the generated scharr mask did not cover the full roi of demosaic but just a tile of it, this lead to crashes.
Due to the way we commit parameters while preparing the pipeline we have to make sure about no tiling while committing params and also make sure in process and the cl variant.
Fixes #15999
For 4.6 and 4.8