-
Notifications
You must be signed in to change notification settings - Fork 251
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
no_auto_scale=True can cause significant clipping and data loss #99
Comments
@atadams I think your use of term "Data loss" is very misleading and your lack of understanding is concerning.
Meaning you're Please understand the difference between what a sensor sees and what a monitor displays, and how the process flow looks like. For VFX artists, the knowledge of reverse flow is important, else you'll produce fake looking images with banding in Histograms. Anomalies in images is
Flagging VFX RAW images disguised as natural images to combat disinformation is a critical role in the field of forensics. Do you agree? To anyone reading, and not trying to be rude, I can suggest few good books or articles on histograms, image processing if you need, please ask. **For VFX artists who make images on monitors and push it into a RAW file, knowing the process is extremely important. ** This is expected behavior of reading an unprocessed RAW data. Please focus on your objective and If you need your specific image discussed, im happy to break it down for you based on latest Sherloq app behavior, which is doing it's job as a forensic app and not an image viewer. Yes, auto scaling is indeed a special case where Again, your image Data is there and not lost my friend. |
@GuidoBartoli I am happy to provide corroborated results from another application. When it comes to the Red Channel and the Camera Sensor Data (and the camera sensor in general) with the 19 photos referenced in the previous issue. Tony presented one of these 19 photos in the previous issue. All 19 photos have specific readings in certain areas on the current version of Sherloq (your recommendation/compromise). We can show this is an issue with the photos. And not the application. We have overwhelming evidence at this point. If you would prefer us share this information publicly. Please let us know we can begin putting everything together. I would just ask for 48-72 hours from the time of notification as it will take some time to put everything together. Link to 19 photos associated with Tony and many of his colleagues. The results of these 19 photos on the current version of Sherloq CAN be corroborated on a different application. Along with significantly more information with camera sensors on all 19 photos. |
JohnConnor01, are you also BobbyoO_ on Twitter / X? "We have overwhelming evidence at this point." I know you may be overwhelmed because your primary experience is trading memecoins, but Canon Digital Photo Professional 4.0 is the only manufacturer created utility for processing CR2 raw files and the results when processing the RAW with the Canon software, (or with just about any other raw software, rawdigger, darktable, etc) it's plain to see that the processing method in the current version of Sherloq is not properly debayering and scaling the non-green channels. |
Indeed, we need to start understanding how some images are unique/anomalous and STOP pushing for changes in the app until the problem goes away. In fact, just based on image fundamentals, requests from Tony is very concerning, as anyone with image processing know how flawed his statements are. In the case of Tony, he does not understand the basics of histogram and how it applies to images. Tony's issue has NO merit anyway you look at it. He is essentially asking for scaling in the guise of "data loss", which is not the case at all. Feels like a repeat of his last flawed request. He just wants his images to show up as they were made on their "monitors" and finding loopholes to get their results at the expense of the app's integrity. The pixels are all there, the data is there, but without scaling the raw data is showing linear ratios, which is what the camera sensor registered. |
This image shared is If you are after image viewer, may I suggest Raw digger, Raw therapee, DarkTable, Canon DPP, Adobe products, or just basic 1party image viewers or irfanview. In summary, these inquiries reveal deficiencies in the foundational principles of image processing, and interestingly coming from the same group (looks like) and dare I say, misleading the forensics app to turn it into yet another image viewer. Request. |
Aren’t you claiming the images have no red channel? It is true that when those images are opened in the current version of Sherloq, the red channels are only zero values. This is explained by my opening comment in this issue. That does not mean the red channel doesn’t exist, only that the red channel is being “zeroed out” when the black levels are subtracted. I welcome Guido to look and any or all of those images as an illustration of this issue I described above. |
This is rude and uncalled for. You've made several comments about me that are really out of line. |
Hello Please re-read my comment. The results of the current version of Sherloq (Gudio’s recommendation/compromise) with the 19 photos linked. Can be corroborated using another application. Ontop of that - significantly more information can be shown on the camera sensors on all 19 photos. Showing how this is an issue with the 19 photos - and not the application. I am simply asking for notice of how that information would like to be released. As it would provide supporting evidence that this version of Sherloq is not reading those 19 photos in error. There are distinct differences when analyzing these specific 19 RAW photos. And other RAW photos from Imagine Resource. And we are happy to illustrate that. And/or provide any/all additional supporting evidence. All RGB intensity values have been pushed to the left with post processing turned down to a minimum. That should have been expected. The compromise was intended to avoid bias results. As stated on the previous issue. This is a sample size of 1 or 2 you are presenting while asking for wholesale changes again. But there are distinct differences with the 19 RAW photos linked. And many other RAW photos we have sampled. Those primarily show up in the Red Channel and/or Red mean vector PCA value. My position is the current version of Sherloq is not reading those 19 photos in error. And I am happy to show the corroborated results with another application. Plus overwhelmingly more evidence of/relating to the camera sensors on all 19 photos. |
This issue happens with every RAW file opened — including the 19 you reference. |
@atadams has raised another issue about clipping with RAW photos on the current version. The RGB intensity values are not getting scaled and showing linear proportions. Which is what you would expect reading the camera censor data. I would argue he is conflating two different things. This was supposed to be the version that avoids biased results. And we greatly appreciate you reaching that compromise. However if his recommendation is accepted and implemented. That will BRING BACK clipping with RAW photos. As highlighted below with two CR2 RAW photos from Imaging Resource. I will go back and find the link to the original photo so this can be replicated. Tony's recommendation will bring back the very problem he said he was trying to solve. Example 1 of RAW Photo being clipped after no_auto_scale=true is removed Example 2 of RAW Photo being clipped after no_auto_scale=true is removed |
@JohnConnor01 looks like those images were already clipped in camera. Both of those images contain direct sunlight reflecting off the water. If the highlights are already clipped, Sherloq isn't going to unclip it. |
The right way to evaluate this is: |
The two examples you provided appear underexposed or overexposed. As Guido pointed out in a previous issue, there is nothing Sherloq should do about that. |
Argument can be made that Tony's 19 images are already anomalous, and the sensor data is revealing the issues. Anyway, the pretense that there is |
Also, could you please refrain from personal comments about me and just address the technical issues? |
You're misreading the statement. John said there's a disconnect between what's being asked, the rationale, and the specific settings you want to introduce to make the assumed problem go away. Do you realize there is no data loss to begin with or is that still not clear to you? |
They aren't my images, but Guido is obviously free to use them in his investigation. The fact that, when opened in Sherloq, the red channels of those images are comprised entirely of zero values is an excellent example of the issue I'm describing. |
Test Photo 1 Test Photo 2 Test Photo 1 on Current Version (No Clipping) Test Photo 1 after no_auto_scale=true is REMOVED (Clipping) Test Photo 2 on Current Version (No Clipping) Test Photo 2 after no_auto_scale=true is REMOVED (Clipping) If the issue is: Clipping with RAW Photos. It appears Tony's recommendation will only increase the frequency at which RAW Photos are clipped while using Sherloq. The compromise again was intended to avoid bias results. |
"the red channels of those images are comprised entirely of zero values is an excellent example of the issue I'm describing" Tony, have you checked other sources to see if they corroborate these results? Because we have. And other sources corroborate these very results. |
Good, then respectfully, let's investigate and not rule images out as a problem. |
Yes, I have checked other sources. The 19 images you refer to all have red channels in every imaging app I've tried. To claim they don't is not reasonable. If you have information that will help Guido make a decision, I suggest you provide it. |
I'll ask again that you stop with the personal attacks. |
Again, those images are underexposed or overexposed. The clipping you are showing isn't due to Sherloq. Guido mentioned this in the issue you opened previously and asked that examples like this not be provided anymore. |
I am happy to provide corroborated results. Given the repeated requests to change software settings and applications in recent history. I am waiting for @GuidoBartoli response on how to publish that information. Separately. I highlighted how your recommendation will only bring back the very problem (clipping with RAW photos) that you say you are trying to solve. Those photos are not being clipped on the current version. But they will be when you apply your recommendation. |
They were clipped (I.e., underexposed or overexposed) when the photo was taken. My recommendation would not change that. |
Where's personal attack in pointing out the fundamental fallacy. Your title says "Data loss" and "clipping", and then you ask for removal of Scaling.
Contrarily, I anticipated a thank you for identifying the fallacy, which I believed would save readers a significant amount of time. |
When you say the issue is my lack of fundamental understanding, it's personal. And I'll ask you again to stop. |
I understand your perspective, but my intention is simply to show that your concern is not a valid argument. We can take the discussion/investigation to my repo to avoid digressions. |
Again. This issue isn't about any specific images. It happens with all images. |
It's not appropriate to be soliciting business on GitHub issues. It might be considered spam. |
The issue you brought up is a non-issue as I pointed out previously, This only highlights, if I may, a fundamental misunderstanding. I'm at a loss for how else to express it. Whenever I point out a technical misunderstanding in your post, you interpret it as a personal attack, which it is not. Hopefully you see read this in a positive light.
You misunderstood and allow me to rephrase. I meant to avoid spam, i can break down your images for potential anomalies instead of shaping Sherloq to render your image exactly as you desired. Use any other image viewer if you dislike gamma (1,1) / no scaling. Is that not an option for you? |
Your assumption is I need your help. I don't. Please refrain from any mention of my abilities or skills and stick to the issue. |
Guys... every time we make a change within the RAW file import a ruckus happens here, you really make me regret leaving this feature inside Sherloq... Let's make this the last adjustment on this, shall we? In 4f89cfa commit, the Again, please do not spam and pull more discussions out of the hat for their own sake. |
@GuidoBartoli this is strange, and you accepted that there is data loss with no scaling = true. As an open-source app owner, you want to see both sides, maintain the integrity of the application and changes based on sound technicals. With scaling on you invariably introduced noise, so can you please introduce noise removal tools please? |
Since Sherloq's inception, "no_auto_scale=true" has never been enabled. It was only last week when it was added. He's just going back to the way it's always been, now without auto changing the white balance or brightness of the image. Sounds like he's leaving the image "as shot" from the camera. Sounds good to me. |
You're advocating for version without Tony's changes good!, then we should go back to (auto_bright=True) and leave everything out! |
I simply removed the
Yes, that's the point. As already explained here, I would again point out that the option to load a RAW file within a forensic analysis program really doesn't make much sense (I don't think even commercial products like Authenticate have it either). |
In v0.89f, adding the
no_auto_scale=True
parameter to theraw.postprocess
in theload_image
function (utility.py
) can cause significant clipping and data loss when opening a RAW file. This is illustrated by view the histogram of this real-world, naturally taken gallery image from Imaging Resource.Red and blue histogram with v0.89f no_auto_scale=True parameter
You can see a large percentage of red and blue pixels have 0 intensity and the image has a strong green tint. I believe the clipping is happening because the black levels are still being subtracted. With other images, the R, G, or B channels can be entirely 0 values.
Alex Tutubalin, the developer of Libraw, has stated, “no_auto_ scale is very special use parameter.”
You can see the results of removing the
no_auto_scale=True
parameter in the histograms below.Red and blue histogram with no_auto_scale=True parameter removed
The image’s color is more balanced and you can see the number of pixels with 0 intensity is significantly reduced.
Recommendation
I recommend removing the
no_auto_scale=True
parameter from theload_image function
inutility.py
.Thank you.
The text was updated successfully, but these errors were encountered: