Great great work!! Here're two feedbacks.
First, I think some variables expressions in the two equations for the MR model on page 21 and 23 are strange. For example, in "total units", "units" is the subscript of "total". At my first glance, I could only notice "total" without "units". However, "units" is something important to show what exactly is the dependent variable. The same applies to "regular price" where I could only notice regular. It also create a consistency problem with another regression equation on page 35 where we use "PP" instead of "PP factor", "Discount" instead of "Discount factor". Although the equations are for different use, we'd better utilize the name of same variable.
Second, the screenshot picture for dynamic input model and event study model is not quite clear after being enlarged. I took better pictures in the attachment. Chao, could you please update them?
Thanks team!
Best,
Helen