biolab / orange3-timeseries

🍊 :chart_with_upwards_trend: Orange add-on for analyzing, visualizing, manipulating, and forecasting time series data.
Other
62 stars 41 forks source link

Fix Yahoo Finance Import #257

Closed rsfutch77 closed 1 year ago

rsfutch77 commented 1 year ago
Issue

Yahoo Finance does not work (see #256)

Description of changes

import new yfinance module

Includes
rsfutch77 commented 1 year ago

So I might have fixed it, but I haven't figured out how to build the whole Orange app yet to test it. My Orange python environment still gives me this error: ModuleNotFoundError: No module named 'yfinance'

PrimozGodec commented 1 year ago

@rsfutch77, thank you for the contribution. It looks good and is working. So, I will merge the fix. Probably, you got a ModuleNotFoundError since the finance module was not installed. You should just call pip install -e . in a directory with this module, and all required packages will be installed.

PrimozGodec commented 1 year ago

And sorry for delay with review.

rsfutch77 commented 1 year ago

Woohoo I just saw this in the release version! Thanks!