Closed truesoni closed 10 months ago
Hi @truesoni, thanks for highlighting this.
The file you indicate is actually deprecated, and cannot be used at all. If anything, we should actually remove this from the source code. Will open up an issue for that, but I'll close this one as won't fix.
ok, thanks for the feedback
Improvement Suggestion: Found an issue: ambiguous variable name 'l'
It's currently on:
Variables named I, O, and l can be very hard to read. This is because the letter I and the letter l are easily confused, and the letter O and the number 0 can be easily confused. Change the names of these variables to something more descriptive.
Example of incorrect code: The code in this example could be misinterpreted to be 0 * 1.08 (zero times one-point-eight).
Example of correct code: This example clarifies that we're multiplying an order variable by another number.