- Notifications
You must be signed in to change notification settings - Fork28
Commitc6f153d
committed
Rethink how to test the hyperbolic functions.
The initial commit tried to test them on trivial cases such as 0,reasoning that we shouldn't hit any portability issues that way.The buildfarm immediately proved that hope ill-founded, and anywayit's not a great testing scheme because it doesn't prove that we'reeven calling the right library function for each SQL function.Instead, let's test them at inputs such as 1 (or something withinthe valid range, as needed), so that each function should producea different output.As committed, this is just about certain to show portabilityfailures, because it's very unlikely that every platform computesthese functions the same as mine down to the last bit. However,I want to put it through a buildfarm cycle this way, so thatwe can see how big the variations are. The plan is to add"set extra_float_digits = -1", or whatever we need in order tohide the variations; but first we need data.Discussion:https://postgr.es/m/E1h3nUY-0000sM-Vf@gemulon.postgresql.org1 parentc6c9474 commitc6f153d
2 files changed
+33
-33
lines changedLines changed: 25 additions & 25 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
454 | 454 |
| |
455 | 455 |
| |
456 | 456 |
| |
| 457 | + | |
457 | 458 |
| |
458 |
| - | |
459 |
| - | |
460 |
| - | |
461 |
| - | |
| 459 | + | |
| 460 | + | |
| 461 | + | |
| 462 | + | |
462 | 463 |
| |
463 | 464 |
| |
464 |
| - | |
465 |
| - | |
466 |
| - | |
467 |
| - | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
468 | 469 |
| |
469 | 470 |
| |
470 |
| - | |
471 |
| - | |
472 |
| - | |
473 |
| - | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
474 | 475 |
| |
475 | 476 |
| |
476 |
| - | |
477 |
| - | |
478 |
| - | |
479 |
| - | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
480 | 481 |
| |
481 | 482 |
| |
482 |
| - | |
483 |
| - | |
484 |
| - | |
485 |
| - | |
| 483 | + | |
| 484 | + | |
| 485 | + | |
| 486 | + | |
486 | 487 |
| |
487 | 488 |
| |
488 |
| - | |
489 |
| - | |
490 |
| - | |
491 |
| - | |
| 489 | + | |
| 490 | + | |
| 491 | + | |
| 492 | + | |
492 | 493 |
| |
493 | 494 |
| |
494 |
| - | |
495 | 495 |
| |
496 | 496 |
| |
497 | 497 |
| |
|
Lines changed: 8 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
154 | 154 |
| |
155 | 155 |
| |
156 | 156 |
| |
157 |
| - | |
158 |
| - | |
159 |
| - | |
160 |
| - | |
161 |
| - | |
162 |
| - | |
163 |
| - | |
164 |
| - | |
165 | 157 |
| |
166 | 158 |
| |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
167 | 167 |
| |
168 | 168 |
| |
169 | 169 |
| |
|
0 commit comments
Comments
(0)