josdejong/mathjs

[Document the] secant function messing up sec unit

Ouverte

#2 095 ouverte le 28 janv. 2021

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)JavaScript (1 314 forks)batch import
documentationhelp wanted

Métriques du dépôt

Stars
 (15 065 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

I believe "sec" is a built-in alias for the time unit seconds. However, if I type "5 sec" into the https://mathnotepad.com (or the evaluate input on https://mathjs.org) the function sec seems to take precedence and throw a TypeError even though it is expecting a unit:

TypeError: Unexpected type of argument in function multiplyScalar (expected: number or Unit or string or boolean or BigNumber or Complex or Fraction, actual: function, index: 1)

Can we improve the parser to avoid this confusion? It seems the unit "sec" is currently unavailable.

Guide contributeur

Direction de recherche
Examinez le code de l'analyseur pour comprendre comment les alias d'unités et les noms de fonctions sont résolus. Identifiez où 'sec' est analysé comme une fonction plutôt que comme une unité. Proposez une modification pour prioriser les alias d'unités par rapport aux noms de fonctions lors de l'analyse des expressions.
Stack technique
javascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70