lightly-ai/lightly

Remove opencv dependency from tests

Offen

#1.477 geöffnet am 15.01.2024

Auf GitHub ansehenContributor Guide
 (14 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (337 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (3.775 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (34 gemergte PRs in 30 T)

Beschreibung

We still have an old opencv dependency in the tests, see here: https://github.com/lightly-ai/lightly/blob/2b215aa5d19b50515f280c1b0181f2f135cbc93f/tests/data/test_VideoDataset.py#L11

We should refactor the test(s) to not rely on opencv and remove opencv from all requirements files.

Contributor Guide

Research-Richtung
Durchsuchen Sie alle Testdateien unter tests/ nach Importen und Verwendungen von 'cv2' oder 'opencv'. Ersetzen Sie die opencv Funktionalität durch Alternativen wie PIL (Pillow) oder numpy. Entfernen Sie opencv aus allen Anforderungsdateien (z.B. requirements.txt, setup.py usw.). Führen Sie die Testsuite aus, um sicherzustellen, dass nichts kaputt geht.
Tech Stack
python
Domain
backendmachine learning
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythonpytest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65