Update Margin Calls 06 Model Structure.html

[kevinwheeler]

Small terminology correction

Description

Related Issue

Motivation and Context

Types of changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] Refactor (non-breaking change which improves implementation)
• [ ] New feature (non-breaking change which adds functionality)
• [x] Non-functional change (xml comments/documentation/etc)

Checklist:

• [ ] My code follows the code style of this project.
• [ ] I have read the CONTRIBUTING document.
• [ ] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

[kevinwheeler]

Hi @AlexCatarino , Thank you for the reply. I don't understand. The current text reads:

"The GetMarginCallOrders method scans the portfolio and the updated data for a potential margin call situation that may get the holdings below zero. The method must return a list of objects that represent the margin call orders. To issue a margin call warning during this method, set the argument of the method to true.

You said "When we get below maintenance margin but not below zero yet, we create the orders"

Shouldn't "zero" be replaced with "maintenance margin" in the text above, since we will create the orders when we get below maintenance margin, and not when we get below zero?

2) If I understand you right, you also say that "total portfolio value" would be a better substitute than "equity". I agree.