Modify `_getExpiredCashValue()` to take the otoken underlying, strike asset, expiry and option type as params instead of taking an Otoken contract. The current MarginCalculator implementation have a lot of repetitive external calls, as an example: - https://github.com/opynfinance/GammaProtocol/blob/master/contracts/MarginCalculator.sol#L49 `getExpiredPayoutRate()` call `_getExpiredCashValue()` that fetch the Otoken properties, but also some of them get called again later https://github.com/opynfinance/GammaProtocol/blob/master/contracts/MarginCalculator.sol#L51 - ....
Modify
_getExpiredCashValue()to take the otoken underlying, strike asset, expiry and option type as params instead of taking an Otoken contract.The current MarginCalculator implementation have a lot of repetitive external calls, as an example:
getExpiredPayoutRate()call_getExpiredCashValue()that fetch the Otoken properties, but also some of them get called again later https://github.com/opynfinance/GammaProtocol/blob/master/contracts/MarginCalculator.sol#L51