dotnet/runtime

Math.Round(value, digits, mode) does not return the correct result

Open

#1643 aperta il 18 apr 2019

Vedi su GitHub
 (13 commenti) (8 reazioni) (1 assegnatario)C# (5445 fork)batch import
Cost:Marea-System.Numericsbreaking-changebughelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

The Math.Round(value, digits, mode) code returns a close, but not correct result most of the time.

The root problem is that the algorithm tries to follow normal mathematical rules to produce the result. However, due to the variable precision in IEEE floating-point numbers, this ultimately causes it to produce a close, but generally incorrect result most of the time and causes it to not accept inputs above a certain range. For example, you can't round double.MaxValue and the larger the input is, the more inaccurate the rounding will be.

These functions should be rewritten to be IEEE compliant and to accept any valid finite floating-point input.

Guida contributor

Direzione di ricerca
Esamina l'attuale implementazione di Math.Round nel codice sorgente di .NET runtime (probabilmente in src/libraries/System.Private.CoreLib/src/System/Math.cs). Comprendi l'algoritmo che tenta di arrotondare usando regole matematiche e identifica dove la precisione in virgola mobile causa imprecisioni. Rivedi le modalità di arrotondamento IEEE 754 (arrotondamento al più vicino, pareggio a pari, ecc.) e la gestione degli input finiti. Proponi una riscrittura che eviti la perdita di precisione per tutti i double finiti, inclusi i casi limite come double.MaxValue. Controlla le discussioni esistenti nell'issue per soluzioni proposte.
Tech stack
csharp
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
.NET internalsIEEE 754 floating point arithmetic
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50