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

The perspective correction module is not working correctly #11158

Closed
Lorenzoeffe opened this issue Feb 16, 2022 · 8 comments
Closed

The perspective correction module is not working correctly #11158

Lorenzoeffe opened this issue Feb 16, 2022 · 8 comments
Assignees

Comments

@Lorenzoeffe
Copy link

Good morning,
I would like to point out that the perspective correction module, in the automatic version, does not seem to work correctly.
In previous versions of DT >3.8, the module worked, but now it corrects the perspective but stretches it.

Auto crop: Indifferent to the end of the test
Lens model: Indifferent to the end of the test
Automatically analyzes structure
Corrects automatically

The perspective is distorted, and stretched

@AlicVB
Copy link
Contributor

AlicVB commented Feb 16, 2022

Although there has been changes in the module for 3.8, the algorithm itself has not changed afaik...
Can I ask you the following :

  • does an image with perspective applied in 3.6 change when opened in 3.8 ?
  • can you take following screenshot, done with the exact following steps :
  1. open an image in 3.6
  2. discard the history
  3. take screenshot n°1 (image uncorrected)
  4. click on "automatic analyze structures"
  5. take screenshot n°2 (with the control lines)
  6. apply the correction
  7. take screenshot n°3 (image corrected)
  8. open same image in 3.8
  9. discard the history
  10. click on "automatic analyze structures"
  11. take screenshot n°4 (with the control lines)
  12. apply the correction
  13. take screenshot n°5 (image corrected)

I know that's quite a big job, but I don't reproduce here, so we need help to understand the root cause...
Thanks !

@AlicVB AlicVB self-assigned this Feb 16, 2022
@Lorenzoeffe
Copy link
Author

Lorenzoeffe commented Feb 17, 2022 via email

@Lorenzoeffe
Copy link
Author

Hello everyone,
screenshots performed.
Missing the initial 1 screenshots with the image just opened, but it matters little for the purpose of detecting the problem.

Greetings
Lorenzo

Image 18 02 2022   155132
Image 18 02 2022   160826
Image 18 02 2022   160821
Image 18 02 2022   155136

@AlicVB
Copy link
Contributor

AlicVB commented Feb 18, 2022

Thanks for the screenshots. Well, there's some other things involved here :

  1. probably the most important : you don't have the same modules applied by default (certainly due to a switch to "filmic" pipe) And that change the resulting image (just see the exposure as the most obvious example, but I suspect other difference too). And of course the resulting analyze of the structres differ (see for example the corner between the right wall and the ground : they are blue in one case and yellow in the other, which means that thir validity differ)
  2. in the second case, it seems you have not choosed the right "apply" button... and so you only correct vertical lines...

I fact I suspect that point 1 is what the cause of the difference you see... To be sure, you can redo the test, but use a jpeg file instead of a raw, so you are quite sure that the base image is the same...
Sorry again to annoy you... I know for sure how boring it is to switch between dt version...

@Lorenzoeffe
Copy link
Author

Hi all.
Regarding point #1, these were "clean" DT installations without me resetting my configuration files.
Regarding point #2, is to be excluded since I didn't press combinations on the keyboard, so the system should correct everything, both vertically and horizontally.
The problem lies as you pointed out in the difference in automatic detection and the consequent application of the correction.

Even with jpeg you get the same results but different. "attached"
Also on the laptop with Mint 20.3, I get the same results. "attached"

Schermata del 2022-02-18 18-15-36
Image 18 02 2022   182453

@AlicVB
Copy link
Contributor

AlicVB commented Feb 18, 2022

The problem lies as you pointed out in the difference in automatic detection

yes... and that point as not changed afaict...
So you confirm that with a (undistorted) jpeg file as source, you get different structures in 3.8 than in 3.6, right ?
Can you by any change share the jpeg file, so I can test in my side and try to reproduce ? (put it somewhere on the web, I fear github apply some recompressing when posting inside issues directly)
Thanks !

@github-actions
Copy link

This issue did not get any activity in the past 60 days and will be closed in 365 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

@github-actions
Copy link

This issue was closed because it has been inactive for 300 days since being marked as stale. Please check if the newest release or nightly build has it fixed. Please, create a new issue if the issue is not fixed.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants