ManimCommunity/manim

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

Open

#3.798 aberto em 9 de jun. de 2024

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Python (1.378 forks)batch import
good first issuenew feature

Métricas do repositório

Stars
 (17.820 stars)
Métricas de merge de PR
 (Mesclagem média 11d 17h) (4 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine a classe `GenericGraph`, provavelmente em `manim/graph.py` (ou similar). Implemente ` getitem ` para lidar com chaves de tupla, retornando a aresta correspondente. Verifique os métodos de recuperação de arestas existentes no atributo `edges`. Garanta a compatibilidade com o comportamento de chaves existente e atualize quaisquer testes ou documentação relevantes.
Pilha de tecnologia
python
Domain
api
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythongraph basicsspecial methods
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75