Generalizable detector + descriptor combos
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Feature
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- python
- Bereich
- computer-vision
Rechercherichtung
Das Issue schlägt eine neue FeaturePipeline-Abstraktion um Detector- und Descriptor-Komponenten vor, die das Verhalten von cv2.Feature2D und kombinierte Algorithmen wie SIFT oder ORB verwendet. Es werden keine Repository-Dateien oder Tests identifiziert. Daher sollen zunächst die vorhandenen Einstiegspunkte für die Feature-Extraktion gefunden und bestimmt werden, wie Detector-Descriptor-Kombinationen derzeit dargestellt werden. Die Fertigstellung sollte eine definierte API für native kombinierte Algorithmen und separate Detector-Descriptor-Paare sowie eine Abdeckung der genannten Verwendungsmuster umfassen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
This has been on the Wishlist for a while. Here's a Gemini idea:
To achieve true polymorphism where any detector $X$ can be cleanly paired with any descriptor $Y$—while also seamlessly supporting combined algorithms (like SIFT or ORB) that handle both natively—you can implement a unified interface layer.
The primary challenge with OpenCV's underlying structure is that everything inherits from cv2.Feature2D in C++, but in Python, the objects are functionally asymmetric: "Detector-only" objects lack .compute(), "Descriptor-only" objects lack .detect(), and combined frameworks implement everything plus .detectAndCompute().
An elegant way to handle this in Python is to treat the Pipeline itself as the polymorphic interface, using a lightweight structural wrapper or duck-typed composition.
Here is a clean, robust architecture that provides a unified API for any combination.
The Polymorphic Pipeline Architecture
By creating a unified container class, you can abstract away whether the underlying execution runs a combined native method or executes two distinct, mismatched components sequentially.
from typing import Protocol, Tuple, Optional, Any
import cv2
import numpy as np
class FeaturePipeline:
"""
A polymorphic wrapper that pairs any keypoint detector with any
feature descriptor, or wraps a single unified extraction algorithm.
"""
def __init__(self, detector: Any, descriptor: Optional[Any] = None):
self.detector = detector
# If no distinct descriptor is provided, assume the detector handles both
self.descriptor = descriptor if descriptor is not None else detector
def detect(self, image: np.ndarray, mask: Optional[np.ndarray] = None) -> list:
"""Extract only the keypoints from an image."""
if hasattr(self.detector, 'detect'):
return self.detector.detect(image, mask)
elif hasattr(self.detector, 'detectAndCompute'):
kp, _ = self.detector.detectAndCompute(image, mask)
return kp
else:
raise AttributeError(f"The configured detector {type(self.detector).__name__} cannot detect keypoints standalone.")
def compute(self, image: np.ndarray, keypoints: list) -> Tuple[list, np.ndarray]:
"""Compute descriptors for a pre-existing list of keypoints."""
if hasattr(self.descriptor, 'compute'):
return self.descriptor.compute(image, keypoints)
else:
raise AttributeError(f"The configured descriptor {type(self.descriptor).__name__} cannot compute descriptors.")
def extract(self, image: np.ndarray, mask: Optional[np.ndarray] = None) -> Tuple[list, np.ndarray]:
"""
Polymorphically extracts both keypoints and descriptors.
Optimises for combined algorithms if available.
"""
# Case 1: Unified native algorithm (e.g., SIFT, ORB)
if self.detector is self.descriptor and hasattr(self.detector, 'detectAndCompute'):
return self.detector.detectAndCompute(image, mask)
# Case 2: Mix-and-match sequential execution (e.g., FAST + FREAK)
keypoints = self.detect(image, mask)
return self.compute(image, keypoints)
How this Solves Polymorphism
This structure allows your client code to treat every single feature extraction mechanism identically, completely removing conditional checks (if isinstance(...)) from your core tracking loops.
1. Using a Native Combined Pair
# SIFT handles both detection and description internally
sift_pipeline = FeaturePipeline(cv2.SIFT_create())
kp, des = sift_pipeline.extract(img)
2. Using a Mix-and-Match Pair
# FAST handles detection, FREAK handles description
fast_freak_pipeline = FeaturePipeline(
detector=cv2.FastFeatureDetector_create(),
descriptor=cv2.xfeatures2d.FREAK_create()
)
kp, des = fast_freak_pipeline.extract(img)
3. Reusing Coordinates for tracking
# Detect features once, then compute them across sequential frames
initial_kp = fast_freak_pipeline.detect(frame_1)
kp_1, des_1 = fast_freak_pipeline.compute(frame_1, initial_kp)
kp_2, des_2 = fast_freak_pipeline.compute(frame_2, initial_kp)
Type Enforcement (Optional Extension)
If you want explicit structural typing flags to catch setup errors before runtime, you can define lightweight Protocol classes to act as interfaces for the sub-components:
from typing import Protocol, runtime_checkable
@runtime_checkable
class DetectorComponent(Protocol):
def detect(self, image: np.ndarray, mask: Optional[np.ndarray] = None) -> list: ...
@runtime_checkable
class DescriptorComponent(Protocol):
def compute(self, image: np.ndarray, keypoints: list) -> Tuple[list, np.ndarray]: ...
By initializing with this approach, the runtime environment doesn't care where the underlying binary resides (core vs. contrib), as long as the bound object satisfies the duck-typed interface behavior.
- Vorherrschende Sprache
- Python
- Sterne
- 224
- Forks
- 31
- Ø Merge
- 1 T. 20 Std.
- Gemergte PRs (30 T.)
- 28
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus petercorke/machinevision-toolbox-python
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
petercorke/machinevision-toolbox-python#152 ·
Maintainer antworten meist innerhalb von 1 Tag
-
ROSBag: RealSense bags have a 4-byte depth-units trailer that is guessed, not parsedEvtl. vergeben @theluckystrike hat das heute übernommen. Offenbug help wanted
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
petercorke/machinevision-toolbox-python#151 ·
Maintainer antworten meist innerhalb von 1 Tag
-
help wanted tech-debt
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
petercorke/machinevision-toolbox-python#150 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 43/100
petercorke/machinevision-toolbox-python#148 ·
Maintainer antworten meist innerhalb von 1 Tag
-
tech-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 30/100
petercorke/machinevision-toolbox-python#147 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in petercorke/machinevision-toolbox-python
Ähnliche Issues
-
bug needs-triage priority:medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
yeongseon/azure-functions-logging-python#583 ·
Maintainer antworten meist innerhalb von 1 Tag
-
documentation needs-triage priority:medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
yeongseon/azure-functions-validation-python#546 ·
Maintainer antworten meist innerhalb von 1 Tag
-
PHP workflow streaming requests buffer the full SSE responseEvtl. vergeben @Winoren hat das heute übernommen. Offenreview: medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
langgenius/dify#44270 · 1 Kommentar · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
P4: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
jeffknupp/association#523 ·
-
bug help wanted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
DarkStarSystems/pcons#229 ·
Maintainer antworten meist innerhalb von 1 Tag