[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 03ad37d7 08/13: Note an arcanum
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 03ad37d7 08/13: Note an arcanum |
Date: |
Mon, 9 May 2022 20:13:18 -0400 (EDT) |
branch: master
commit 03ad37d72e26394e984ff2b2e93e65dcda1752d7
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
Note an arcanum
There's a reason why this assertion has long been made for sales_load_:
LMI_ASSERT(0.0 <= sales_load_);
but not for premium_load_. Writing one for premium_load_ and commenting
it out is less obscure than omitting it.
The reason is that a "discount" was to be given for a certain subplan to
reflect lower commissions. Notionally, the "sales load" represents
commissions, but in practice it was already less than the commission
reduction, so it was expeditious to set a negative rate for this load.
---
ihs_avmly.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/ihs_avmly.cpp b/ihs_avmly.cpp
index f1cbb700..d38b2648 100644
--- a/ihs_avmly.cpp
+++ b/ihs_avmly.cpp
@@ -1529,6 +1529,8 @@ currency AccountValue::GetPremLoad
target_portion * YearsPremLoadTgt
+ excess_portion * YearsPremLoadExc
;
+// There actually exists a product with a negative premium load.
+// LMI_ASSERT(0.0 <= premium_load_);
sales_load_ =
target_portion * YearsSalesLoadTgt
- [lmi-commits] [lmi] master updated (9e5a09bf -> 3903a0c3), Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master a1533521 01/13: Replace a history command, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 3903eed7 03/13: Include appropriate headers, and say why they're included, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master d9c5f0d5 04/13: Use a local alias to shorten expressions, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 3d69769b 05/13: Demonstrate that a default comparison tolerance works well, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 4d7fe889 07/13: Use rate_times_currency(), Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 03ad37d7 08/13: Note an arcanum,
Greg Chicares <=
- [lmi-commits] [lmi] master a67210ec 10/13: Better explain the purpose of each block of unit tests, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master ee272a29 02/13: Suppress certain GTK diagnostics, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master c768da7e 06/13: Refactor, almost, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 35eb8545 09/13: Prefer a calculation with less roundoff error, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master c53ea12a 11/13: Reorder some statements in a unit test, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 8b6d6dd8 12/13: Say DECIMAL_DIG wherever it's meant, Greg Chicares, 2022/05/09
- [lmi-commits] [lmi] master 3903a0c3 13/13: Record speed measurements, Greg Chicares, 2022/05/09