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
Spot Removal not working properly #7026
Comments
@Hombre57 - I believe it was you who led this change a few years ago, can you look. Thanks. I confirm the bug found by @PDeb108 The problem was found during the challenge to test "lacam16n2" but it seems that this malfunction occurs whatever the branch, and probably dates back a long time Here the file Raw, and the pp3 I've tried several changes that make it work less badly. It seems that: crop, Lens profile, Input profile have a strong impact. It works only with "neutral". I used as a basis the current work of the "cbdlciecam" #7017 branch because as @PDeb108 uses Ciecam and certain functions which created artifacts, I preferred to start from a healthier base to eliminate possibilities (it seems that Ciecam16, Wavelets or LA are not involved... to check) This looks like a pipeline problem, plus this type of process is not in my area of expertise I create a new branch "spot_cbdlciecam" Jacques |
Since spots are located at fixed positions in the initial image, entering the edit mode disable all transform tools that you could have already set, so you can preview the image in its initial state. That's what the rtengine::TweakOperator has been created for. See line 866 of spot.cc for all tools forced to neutral. The last group on line 883 is for better performance, because the spot tool is already time consuming if there are numerous big spots. If new tools are added to RT, this list might have to be updated (probably the local editing tools should be disabled here). Tone curves are not disabled IIRC, and that's perhaps why it looks over exposed. Keep in mind that spots should be removed in the beginning of your editing process to avoid this kind of problem, even with a temporary exposure compensation. There's no magic solution here I'm afraid. |
Thank you for the explanation. I think to improve the operation we should add to Spot.cc (tweaksparams)
And and perhaps others parameters. Jacques |
Thank you for the explanation. As a user, I don't understand the technicalities you have mentioned, however, I gather, that spot-removal is a resource-intensive process and should be done before doing anything else, much like setting the white balance first. Doing it later might even cause crashes. I am yet to try it on any photos as yet. will give you a feedback Might I suggest that the module be moved from the detail group to the exposure group to focus on the need to use it early in the work process |
@Desmis I've not installed any IDE since I changed laptop, so could you please do that change ? I wouldn't reset So the patch would look like : pparams.locallab.enabled = false;
pparams.toneCurve.hrenabled = false; // not sure for this one, it could be useful for ExpComp w/o performance penalty
pparams.toneCurve.histmatching = false; No need to reset the entire @PDeb108 Tools are sorted into tabs depending on their functionality, and tabs are organized (somewhat) by their usage count. That's why RAW is almost the last tab while being the first thing happening in the process pipeline. So I think Spot is in the right tab. |
That is perfectly logical. Thanks for the input. BTW, although I created the issue, I am not sure whether it should be closed by me or not. Someone more knowledgeable may update the status. Thank You |
Closing because the behavior is expected. |
The problem I am facing in RT is using the spot removal module. I am working with it at the finishing stage of the work. What is happening is that as soon as I click the button on the right to activate the spot (marked with a red circle in the bottom frame). The exposure of the entire picture increases to the point when I am unable to see the spot. The top frame is when I have not activated the spot creation button, and you can see the spot.
This is when I am using the most recent dev version of RT (RawTherapee_7023merge_5.10-182-g918ef5e5c_win64_release)
My system is Intel I5/16GB/Intel graphics processor/Windows 11
The situation is similar even when I use the stable release 5.10
Below is the nature of the problem
The text was updated successfully, but these errors were encountered: