ManimCommunity/manim

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

Fermée

#3 798 ouverte le 9 juin 2024

 (2 commentaires) (0 réaction) (0 personne assignée)Python (1 378 forks)batch import
good first issuenew feature

Métriques du dépôt

Stars
 (17 820 étoiles)
Métriques de merge PR
 (Merge moyen 90j 10h) (31 PRs mergées en 30 j)

Description

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

Guide contributeur