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

5_Bildanalyse Übung 2 l_d Opening #4

Closed
FelixVolo opened this issue Jul 22, 2021 · 11 comments
Closed

5_Bildanalyse Übung 2 l_d Opening #4

FelixVolo opened this issue Jul 22, 2021 · 11 comments

Comments

@FelixVolo
Copy link
Contributor

Ich weiß zwar nicht wieso das so ist, aber folgende Mängel sind mir an der Lösung aufgefallen:

  • Das Eingabebild ist falsch. Es fehlen oben rechts Pixel.
  • Das Ergebnis beim Opening ist falsch. Die Form oben links beim Ergebnis ist mit diesem Filterkern eigentlich nicht möglich... und generell sieht das Ergebnis bei mir wenn ich es per Hand mache vollkommen anders aus.
@FelixVolo
Copy link
Contributor Author

Komischerweise bekomme ich das richtige Ergebnis, wenn ich für die Dilatation folgenden Kern anwende:

[0, 0, 0],
[1, 1, 0],
[0, 1, 0]

@FelixVolo
Copy link
Contributor Author

#5

@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

Kann es sein, dass OpenCV hier eine Faltungs-Operation durchführt? Dann würde die Spiegelung ja passen oder nicht?

@FelixVolo
Copy link
Contributor Author

https://stackoverflow.com/questions/50668212/opencvs-dilate-differs-from-scipy-matlab

Laut der Antwort hier scheint es wohl so.

@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

Danke! Da sieht man, dass sich ein Code review lohnt :) Wäre vermutlich sonst nie aufgefallen...

@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

Ich schließe mal, weil die Antwort gefunden wurde. Der Pull Request wird natürlich akzeptiert!

@TimoK93 TimoK93 closed this as completed Jul 22, 2021
@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

Habe deinen Pull Request nachbearbeitet: Jetzt so in Ordnung?

@FelixVolo
Copy link
Contributor Author

Nein nicht ganz. Der Kernel wurde zwar transponiert, was zum gewünschten Ergebnis führt, aber erst sollte die Erosion mit dem ursprünglichen Kernel stattfinden und dann die Dilatation (die sogar weg ist) mit den transponierten Kernel.

@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

Au, sorry. Da habe ich zu viel gelöscht... Sollte jetzt alles soweit drin sein.

@FelixVolo
Copy link
Contributor Author

Habe das mit einer neuen Pull Request korrigiert

@TimoK93
Copy link
Collaborator

TimoK93 commented Jul 22, 2021

zu spät :)

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

2 participants