Skip to content

Remove ib-trading-mode - #668

Open
AlexCatarino wants to merge 1 commit into
QuantConnect:masterfrom
AlexCatarino:bug-270-remove-ib-trading-mode
Open

AlexCatarino wants to merge 1 commit into
QuantConnect:masterfrom
AlexCatarino:bug-270-remove-ib-trading-mode

Conversation

@AlexCatarino

@AlexCatarino AlexCatarino commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

The IB brokerage now infers the trading mode from ib-account (QuantConnect/Lean.Brokerages.InteractiveBrokers#270), so the ib-trading-mode internal input is no longer needed.

The modules-1.14.json change is git-ignored and must be applied to the CDN copy; the diff is in the comment below. Engine images that predate that brokerage PR still require the key.

🤖 Generated with Claude Code

The IB brokerage now infers the trading mode from ib-account
(QuantConnect/Lean.Brokerages.InteractiveBrokers#270), so the CLI no longer
needs to set it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@AlexCatarino

Copy link
Copy Markdown
Member Author

Diff of lean/modules-1.14.json against the CDN copy:

--- a/lean/modules-1.14.json
+++ b/lean/modules-1.14.json
@@ -69,28 +69,6 @@
                     ]
                 },
                 {
-                    "id": "ib-trading-mode",
-                    "type": "internal-input",
-                    "value-options": [
-                        {
-                            "condition": {
-                                "dependent-config-id": "ib-account",
-                                "pattern": "^df|^du|^di",
-                                "type": "regex"
-                            },
-                            "value": "paper"
-                        },
-                        {
-                            "condition": {
-                                "dependent-config-id": "ib-account",
-                                "pattern": "^f|^i|^u",
-                                "type": "regex"
-                            },
-                            "value": "live"
-                        }
-                    ]
-                },
-                {
                     "help": "Your Interactive Brokers username",
                     "id": "ib-user-name",
                     "input-method": "prompt",

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