small changes and todos for hedge ratio #315
Merged
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.
hedge_ratio seems mostly fine, but please check out the todos in the pull request
Some clean-up for efficiency and transparency is needed in hedge_calculator.
Pandas offers functions to solve some custom problems quickly.
I suggest the following for getting hedge ratios into a frame:
[1] fill pre-allocated df with estimates on days of estimation
[2] upsample to original businness day frame with asfreq() and 'ffill'
[3] then use .shift(1)
You would need neither date_weekend nor the lambda function date_adjustment.
In the test file you should add test of correct hedge ratio by comparing one or more ratio with a
regression result generated outside the functions.