voigtstefan
commented
4 months ago Comments by @christophscheuch (06-06-2024):
- zeile 111: FYI wir haben CRSP data since beginning of 1960, da kann also eine discrepancy herkommen
- zeile 154: das head() muss weg oder?
- zeile 179: wir haben daten bis ende 2022, vl möchtest du auch nur bis 2016 gehen for consistency? wer weiß ob deren results länger halten
- zeile 254: da wieder head()
- zeile 265 - 267: please use j instead of i (aka imaginary unit e.g 1i is imaginary)
- zeile 388: könntest tidyfinance package nehmen, das hat weniger dependencies als tidyverse
Not finalized yet but opened PR to ease feedback collection