josdejong/mathjs

UnaryMinus around ConstantNode not mached by "c" in simplify

Open

#1.406 aberto em 15 de fev. de 2019

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (1.298 forks)batch import
featurehelp wanted

Métricas do repositório

Stars
 (13.832 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

A unaryMinus wrapped around a ConstantNode is matched by "v", not "c", in the simplify function. While this matches the documentation, it is not what is expected for a "constant" and a "non-constant". Other single variable functions with a constant node are simplified to their result, and thus can be matched with "c". If existing compatibility presents an issue, this could be enabled as an option of the simplify function that defaults to the old behavior or by adding two new simplification selectors.

Guia do colaborador

Direção de pesquisa
Examine a função simplify no código fonte do mathjs para entender como os nós unaryMinus são correspondidos. Considere adicionar uma opção para tratar unaryMinus de constantes como constantes.
Pilha de tecnologia
javascript
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75