View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007529 | Taler | wallet (Android App) | public | 2022-12-22 15:53 | 2023-09-23 13:09 |
| Reporter | Windfisch | Assigned To | grote | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | unable to reproduce | ||
| Product Version | 0.9 | ||||
| Target Version | 0.9.3 | Fixed in Version | 0.9.3 | ||
| Summary | 0007529: Withdrawing 13.37 or 133.7 results in rounding errors. | ||||
| Description | The android wallet displays fees and amounts wrong. 133.7 KUDOS become 133.699999999. | ||||
| Steps To Reproduce | 1. Use the android wallet 2. Withdraw 13.37 coins from an exchange with (0.01, 0.10, 1.00, 10.00, 100.00, 1000.00) as denominations, or withdraw 133.7 coins from taler demo exchange. 3. In the confirmation screen for the demo wallet, I get: - "Withdraw: 133.4 KUDOS" (wrong, afaik it should be 133.5 due to the 0.2 fees) - "Selected amount: 133.69999999 KUDOS" (wrong!) - "Fees: 0.29999999 KUDOS" (wrong) | ||||
| Additional Information | Sounds like a floating point issue to me. | ||||
| Tags | No tags attached. | ||||
| I've just re-confirmed that this is not a wallet-core issue (by doing the demo withdrawal with the CLI). The wallet internally also doesn't ever use floating point numbers for amounts. So this is a UI issue. | |
| When withdrawing 137.7 KUDOS from exchange.demo.taler.net, the Withdraw UI shows me Withdraw 137.2 KUDOS Chosen Amount: 137.37 KUDOS Fee: -0.17 KUDOS which looks ok. Any other way to reproduce this? | |
| We can't reproduce this in the Android UI (anymore?), and we don't use any floating point computation in wallet-core. So this should be fixed (and I'm not sure where it occurred in the first place). | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2022-12-22 15:53 | Windfisch | New Issue | |
| 2022-12-22 15:53 | Windfisch | Status | new => assigned |
| 2022-12-22 15:53 | Windfisch | Assigned To | => Florian Dold |
| 2023-01-08 09:20 | Christian Grothoff | Product Version | => 0.9 |
| 2023-01-08 09:20 | Christian Grothoff | Target Version | => 0.9.2 |
| 2023-02-06 12:39 | Florian Dold | Assigned To | Florian Dold => |
| 2023-02-06 12:39 | Florian Dold | Assigned To | => grote |
| 2023-02-06 12:39 | Florian Dold | Note Added: 0019767 | |
| 2023-02-21 13:02 | grote | Note Added: 0019896 | |
| 2023-02-24 13:58 | Florian Dold | Target Version | 0.9.2 => 0.9.3 |
| 2023-02-24 13:58 | Florian Dold | Status | assigned => feedback |
| 2023-04-04 16:08 | Florian Dold | Status | feedback => informing |
| 2023-04-04 16:08 | Florian Dold | Resolution | open => unable to reproduce |
| 2023-04-04 16:08 | Florian Dold | Note Added: 0020014 | |
| 2023-09-23 13:07 | Christian Grothoff | Fixed in Version | => 0.9.3 |
| 2023-09-23 13:09 | Christian Grothoff | Status | informing => closed |