ManimCommunity/manim

Add tuple key support in `GenericGraph.__getitem__()`

Open

#3.798 geöffnet am 9. Juni 2024

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (1.378 Forks)batch import
good first issuenew feature

Repository-Metriken

Stars
 (17.820 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11T 17h) (4 gemergte PRs in 30 T)

Beschreibung

Description of proposed feature

vertices = [1, 2, 3, 4]
edges = [(1, 2), (2, 3), (3, 4), (1, 3), (1, 4)]
g = Graph(vertices, edges)
assert isinstance(g[1], Dot)  # This is OK
assert isinstance(g[(1, 2)], Line)  # But this will raise a KeyError

imo This is confusing. Why not add a support for edge mapping?

How can the new feature be used?

Every time people want to get or set a edge, just write g[(1, 2)] instead of g.edges[(1, 2)].

Contributor Guide

Research-Richtung
Untersuche die `GenericGraph` Klasse, wahrscheinlich in `manim/graph.py` (oder ähnlich). Implementiere ` getitem `, um Tupel Schlüssel zu verarbeiten und die entsprechende Kante zurückzugeben. Überprüfe vorhandene Kantenabrufmethoden im `edges` Attribut. Stelle Abwärtskompatibilität mit dem bestehenden Schlüsselverhalten sicher und aktualisiere relevante Tests oder Dokumentation.
Tech Stack
python
Domain
api
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythongraph basicsspecial methods
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75