Skip to content

Include contract multiplier in serialized option order value#9405

Open
sqsge wants to merge 1 commit intoQuantConnect:masterfrom
sqsge:bug-8447-order-value-multiplier
Open

Include contract multiplier in serialized option order value#9405
sqsge wants to merge 1 commit intoQuantConnect:masterfrom
sqsge:bug-8447-order-value-multiplier

Conversation

@sqsge
Copy link
Copy Markdown

@sqsge sqsge commented Apr 15, 2026

What

  • fix serialized/exported value for option orders in ReadOrdersResponseJsonConverter
  • add a regression test covering option order serialization value
  • update option order JSON fixtures to match multiplier-aware serialized output

Why

Issue #8447 reports that exported order value omits the option contract multiplier, so option orders are serialized with quantity * price instead of quantity * price * contract multiplier.

How

  • keep the fix local to serialization
  • preserve existing behavior for non-option orders
  • when serializing option orders, compute value as order.Value * contractMultiplier

Testing

  • dotnet build Tests/QuantConnect.Tests.csproj --no-restore -v minimal
  • dotnet test Tests/QuantConnect.Tests.csproj --no-build --no-restore --filter FullyQualifiedName=QuantConnect.Tests.Common.Orders.ReadOrdersResponseJsonConverterTests.SerializesOptionOrderValueUsingContractMultiplier -v minimal
  • targeted regression test passed locally: 1 passed, 0 failed

@sqsge sqsge changed the title Fix serialized option order value contract multiplier Include contract multiplier in serialized option order value Apr 15, 2026
@sqsge sqsge marked this pull request as ready for review April 15, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant