josdejong/mathjs

Error comparing temperature units with offset

Aperta

#3134 aperta il 21 gen 2024

Vedi su GitHubGuida contributor
 (11 commenti) (0 reazioni) (0 assegnatari)JavaScript (1314 fork)batch import
bughelp wanted

Metriche repository

Star
 (15.065 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Hi,

In versiones newer than 11.4.1 the comparison of temperatures is incorrect between some temperature units when an offset is involved.

lowTemp = 9 degC;
highTemp = 10 degC;

lowTemp         < highTemp          # true
lowTemp to degF < highTemp          # false but should be true
lowTemp to K    < highTemp          # false but should be true
lowTemp to degR < highTemp          # false but should be true
lowTemp to K    < highTemp to degF  # false but should be true
lowTemp to degR < highTemp to degF  # false but should be true

This was discussed in #2518 but this might need to be a separate issue.

Guida contributor

Direzione di ricerca
Indaga come mathjs gestisce i confronti tra unità, concentrandoti sulle unità di temperatura con offset (es. Celsius, Fahrenheit). Esamina la funzione di confronto e la logica di conversione delle unità per identificare dove l'offset viene applicato in modo errato.
Tech stack
javascript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70