[ENH] Remove yfinance as a dependency and implement data_loader#721
[ENH] Remove yfinance as a dependency and implement data_loader#721Shuvam586 wants to merge 1 commit intoPyPortfolio:mainfrom
Conversation
|
instead of editing tickers mentioned in the also removed |
fkiraly
left a comment
There was a problem hiding this comment.
Nice, thanks!
May I request to not use data downloaded via yfinance from Yahoo services at all? This is due to terms of use, we should not distribute data from Yahoo services at all in the repository or package.
Could you instead use similar data? Either completely randomly generated (Brownian motion random walk or similar, with same column names and time index), or taking some inspiration from the actual data in how you randomize - but it cannot be the exact values.
Closes #716
pypfopt.dataloaders for stock prices and market caps.