search

mcdallas / wallstreet

Real time stock and option data.
MIT License
1.33k stars 201 forks source link

Fixed riskfree function, added m2 #39

Closed Mr-Elysium closed 2 years ago

Mr-Elysium commented 2 years ago

Fixed riskfree() added m2 Now the correct riskfree function should be returned

ishaantaylor commented 2 years ago

@mcdallas what's the blocker to merge this?

Mr-Elysium commented 2 years ago

I do not know what you mean, I just saw that the m2 was nog yet implemented and I implemented it.

mcdallas commented 2 years ago

@ishaantaylor looks like there were changes to the treasury page recently https://home.treasury.gov/developer-notice-xml-changes we have to update the parser to pull the data from those XML feeds instead

Mr-Elysium commented 2 years ago

I was not aware of that, if need be, I am willing to rewrite the function to pull from xml data, actually yes, I will do that.

mcdallas commented 2 years ago

closing in favor of https://github.com/mcdallas/wallstreet/pull/41