feature: add clear street brokerage integration - #666
Open
Romazes wants to merge 1 commit into
Open
Conversation
- add the clear street options to the live deploy commands
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds Clear Street to the live deployment commands. The change is the regenerated
README.md, because the module itself is described inmodules-1.14.json, which this repository does not track.The module is a brokerage only. Clear Street has no market data websocket and no historical bars endpoint, so it declares no
data-queue-handler, nohistory-providerand nodata-downloader. It asks for three values:The API address is an internal input, so nobody is asked for it. The poll interval is optional and falls back to 3000 milliseconds, because Clear Street sends no order events and the brokerage reads the orders on a timer.
Related PR(s)
N/A
Related Issue
N/A
Motivation and Context
Without this,
lean live deployandlean cloud live deploydo not offer Clear Street, so the plugin cannot be used from the CLI.Requires Documentation Change
The brokerage needs its own page under the live trading brokerages section, like the other brokerages have.
How Has This Been Tested?
cli_brokeragesandcloud_brokerageseach resolve one Clear Street module, andget_configs_for_optionsreturns the access token, the account id and the poll interval for a local run, and the first two for a cloud run.README.mdwas regenerated withscripts/readme.py. The change is 11 added lines and 2 replaced lines, all of them Clear Street.Types of changes
Checklist:
bug-<issue#>-<description>orfeature-<issue#>-<description>