Skip to content
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

Add a function for downloading 10 year treasury data to use as a benchmark #134

Closed
wants to merge 2 commits into from

Conversation

benmccann
Copy link
Contributor

See #132

The 10 year treasury should be used as the benchmark. E.g. Investopedia states The 30-year Treasury used to be the bellwether U.S. bond but now most consider the 10-year Treasury to be the benchmark..

Besides the 10-year rate being the more common benchmark, it's also far easier to get better data for. Beginning February 18, 2002, Treasury ceased publication of the 30-year constant maturity series. On February 9, 2006, Treasury reintroduced the 30-year constant maturity. These 4 years of missing data makes it difficult to use as a benchmark. Right now zipline's logging is complaining vociferously about this missing data.

@twiecki
Copy link
Contributor

twiecki commented Apr 16, 2013

Looks good to me. Thanks!

@ehebert
Copy link
Contributor

ehebert commented Apr 17, 2013

Thinking of the best way to wire up this data so that it is used by the risk module.

One option is to splice it in to loader.dump_treasury_curves so that function calls this new method, as well.

@twiecki @benmccann any other ideas there?

@benmccann
Copy link
Contributor Author

Yeah, I wasn't quite sure either. I wonder if we need the existing yield curve data at all? I don't understand how it's chosen to use 1-month vs. 3-month vs. 10 yr. vs. 30 yr. treasuries at the moment. I wonder if always just using 10 yr. is better?

@twiecki
Copy link
Contributor

twiecki commented Jul 19, 2013

@ehebert Any reason we can't pull this in? Then we could change it zipline to use 10 yr instead.

@ghost ghost assigned ehebert Jul 19, 2013
@ehebert
Copy link
Contributor

ehebert commented Jul 19, 2013

No reason not to pull in the precursor, I'll do git am now, since there's a merge conflict reported.

As for pulling into to risk, still hot on the trail of getting the risk module aligned with the risk answer key, and holding off on changing any behavior in the risk module, including treasury choice until all the tests in test_risk.py are reading from the key. (Since we'll have to go into the spreadsheet and change the treasury column etc.)

@ehebert
Copy link
Contributor

ehebert commented Jul 19, 2013

Merged via 2751e98

@ehebert ehebert closed this Jul 19, 2013
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.

None yet

3 participants