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.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create PostPairsTrading.md #11
base: 2-basic-study-of-pair-trading-using-linear-regression
Are you sure you want to change the base?
Create PostPairsTrading.md #11
Changes from 14 commits
36efa3b
70bc399
33eaf4c
143b253
410b768
26301b1
7ea8bd4
74fdca3
c3fdb67
22b9ec8
855277f
49a0bd6
339e18a
a0b7604
9d884cb
2bea233
5c786d3
248adfd
1fa999a
569b53d
55f3343
e9184dc
cdfad08
97f266a
6b81f23
7fc14fd
96bcbaa
579baa1
4b5a104
79fb6ee
ab9050c
d9efeb2
7b4651b
4c7203c
dc9a86b
5e060f2
4348232
f7a2465
2ddc2a2
e6c653f
afbefc0
80fcc3b
83be685
a19e3cc
a57a3ac
e5710aa
0f39e45
1ab6f8e
786bb8e
03c6880
06224b5
4605d9e
135c393
41733c2
c566387
ff1dc17
dc68da1
f1b0bf4
d815f26
9a04521
76561a8
05aa9e1
9024579
b0192f8
d3a5fd3
0ac45dc
636eff5
fc66279
e47d499
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we should add a new section, which supplies a very brief intro to the components of the classical Tick Architecture. I would end up clarifying the next section. First, we'll identify cointegrated pairs. Then, we'll produce the Pairs Trading model to calculate spreads. Finally, we'll exploit such model from a real-time component and will visualise its outcomes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now, this section becomes "Identifying cointegrated indexes" or something like that...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that
crossedList
andsymList
also need to be polished, we could be more 'q' idiomatic.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure if I understand what you mean by this. Should I just join this 2 lines together? and leave it with something like:
crossedList: x cross x: exec distinct sym from superTab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it ok to keep pairs with repeated assets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, cointegration is 2 directional, meaning that cointegration is different between A-B and B-A, so depends on the direction you are taking you will focus on one set of pairs or the opposite. In order to keep it generic I think this is ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I miss the
pub
method from the RPT component itself, where you accumulate on theprices
table. Which component is responsible for cleaning the published spreads?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the second reference to a future Kalman filter post in this section. I think there's some redundancy with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is true but I am figuring out how to put it. Because next post will explain two different things regarding pairs trading:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be unfinished...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Complete the "special thanks", don't forget Javier.