ManimCommunity/manim

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

Open

#3 798 ouverte le 9 juin 2024

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Python (1 378 forks)batch import
good first issuenew feature

Métriques du dépôt

Stars
 (17 820 stars)
Métriques de merge PR
 (Merge moyen 11j 17h) (4 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