Resolve Pandas FutureWarnings

[JuliaLWang8]

Fixes #1739 :

• FutureWarning deprecation warning for pandas resolved by updating fillna and iloc in the reconstruct_intervals_batch function.
• Added a unit test to tests/ticker.py in TestTickerHistory called test_reconstruct_intervals_batch, also validated against current output.

[JuliaLWang8]

I also had to manually pip install packages not in requirements.txt for testing: requests_cache, and requests_ratelimiter. Is there a reason those aren't included in the reqs?

[ValueRaider]

Requests caching and rate-limiting are optional for users, and while required for tests I figure developers are smart enough to install.

Btw the requirement is to avoid tests spamming Yahoo. So not strictly required, but highly recommended.

[bot-unit]

Requests caching and rate-limiting are optional for users, and while required for tests I figure developers are smart enough to install.

Btw the requirement is to avoid tests spamming Yahoo. So not strictly required, but highly recommended.

Maybe we should add this requirements with pip[optional]? docs stockoverflow

[ValueRaider]

Maybe we should add this requirements with pip[optional]? docs stockoverflow

Fine with me. I propose naming pip install yfinance[nospam]

[JuliaLWang8]

Maybe we should add this requirements with pip[optional]? docs stockoverflow

Fine with me. I propose naming pip install yfinance[nospam]

Should we include scipy as an optional req too? I noticed it was also purposely excluded in base.py#L872. If we want it separate, both pandas_datareader and scipy could be under yfinance[optional] or something:

extras_require={
        'nospam': ['requests_cache>=1.1.1', 'requests_ratelimiter>=0.4.2'],
        'optional': ['scipy>=1.10.1', 'pandas_datareader>=0.10.0'],
},

[ValueRaider]

Should we include scipy as an optional req too? I noticed it was also purposely excluded in [base.py#L872]

I don't mind, but if yes then I propose naming yfinance[repair], because only used by price repair.

pandas_datareader

No

Wait for this optional dependencies discussion to resolve.

[JuliaLWang8]

I don't mind, but if yes then I propose naming yfinance[repair], because only used by price repair.

pandas_datareader

No

Sounds good, I can add the discussed extra dependencies to setup.py. Should I make a new PR for that or just do it here?

Also with respect to documentation, does this need to be added somewhere like the readme?

[ValueRaider]

Create a new PR, reference this for background discussion. Should have done this earlier tbh.

Update README too, maybe move install section to top, and replace requirements with link to file.