ManimCommunity/manim

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

Open

#3798 aperta il 9 giu 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Python (1378 fork)batch import
good first issuenew feature

Metriche repository

Star
 (17.820 star)
Metriche merge PR
 (Merge medio 11g 17h) (4 PR mergiate in 30 g)

Descrizione

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)].

Guida contributor

Direzione di ricerca
Esamina la classe `GenericGraph`, probabilmente in `manim/graph.py` (o simile). Implementa ` getitem ` per gestire chiavi tuple restituendo il corrispondente arco. Controlla i metodi di recupero archi esistenti nell'attributo `edges`. Assicura la compatibilità all'indietro con il comportamento delle chiavi esistenti e aggiorna eventuali test o documentazione pertinenti.
Tech stack
python
Dominio
api
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythongraph basicsspecial methods
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75