Open hovinh opened 1 year ago
I just realized this is documented in the API. Still, if the case of positive + negative can be an extension, I'm happy to help :)) Or, if the data is missing some of the date, we can populate them with value from the day before.
@hovinh That's a great idea. I think there should be more messaging around missing data, or missing tickers. Do you want to make a branch?
Sure, I could start working on it next week and update this thread. I will do a PR once it is ready.
Hi @bkrayfield , after reviewing the API, I would like to clarify the contribution:
Single.__init__()
is the "entry" function for model_data to be processed by any model of choice, hence an additional Single._assert_data_quality()
to be invoked within Single.__init__()
can be used for data check. This function can check for data availability and output warning message based on given parameters model_data
, event_date
, event_window
, estimation_size
, buffer_size
.Single._assert_data_quality()
utils.Logger
: a class to handle logging.Could you help confirm my approach and let me know any additional implementation?
Hello,
I wonder if it's a good idea to extend this package to cover the data preprocessing, i.e. transform data into ready-to-consumed before feeding it into this package.
To elaborate further, when I research event study, I realize there are two different ways to compute daily stock return, that are:
Also, when I work on a use case where data could span in both negative and positive zones, I must develop a new variant for the first approach to handle corner cases dealing with 0. I'm new to this and am unsure if an event study can be applied to this case.
Please let me know if this idea is good and if I can contribute to this feature. Thank you :))