search

kamilkrukowski / sec-sentiment-pred

0 stars 1 forks source link

stock-prediction branch setup instructions #7

Open kamilkrukowski opened 1 year ago

kamilkrukowski commented 1 year ago

I don't think there's instructions written anywhere for generating or obtaining TIKRS_Data.pkl, although get_strategy_annual_return will crash if the file does not exist.

Could we have get_strategy_annual_return generate TIKRS_Data.pkl if it needs to?

kamilkrukowski commented 1 year ago

Update:


    company_df = self.historical_data[tikr]
KeyError: 'AAPL'```

Even though I have a ```TIKRS_Data.pkl``` I can't run the eval function anymore due to uncaught error-handling.
yyitingli commented 1 year ago

Update:

    company_df = self.historical_data[tikr]
KeyError: 'AAPL'```

Even though I have a ```TIKRS_Data.pkl``` I can't run the eval function anymore due to uncaught error-handling.

it work with lower case 'aapl'.

kamilkrukowski commented 1 year ago

How about abnb? Seems to not have been updated to include Neal's newer tikrs even if turned lower-case.

Issue stands, we'll need a new TIKRS_Data.pkl whenever possible to evaluate our current dataset, and we'll need a pipeline because It'll have to be regenerated every time Neal adds data.