search

ycroissant / plm

Panel Data Econometrics with R
GNU General Public License v2.0
49 stars 13 forks source link

`detect.lindep.data.frame()` does not pass on the `suppressPrint` parameter #11

Closed joachim-gassen closed 3 years ago

joachim-gassen commented 3 years ago

Hi there:

If I am not mistaken, detect.lindep.data.frame() does not pass on the suppressPrint parameter to detect.lindep.matrix(). At least for me, this causes the parameter to have no effect when calling detect.lindep() with a dataframe as first argument and suppressPrint = TRUE. The same seems to be the case for detect.lindep.plm() (untested).

II am talking about lines 205 and 215 of detect_lin_dep_alias.R. Please feel free to correct me if I am off.

Thanks for maintaining this great package!

Joachim

tappek commented 3 years ago

Thank you for reporting and the kind words about our package! Good catch and very precise report.

I have just fixed this in our development repository rev. 1298, https://r-forge.r-project.org/R/?group_id=406. Woudl you mind giving it try? install.packages("plm", repos="http://R-Forge.R-project.org")

In other news you might want to try the fast mode (enable options("plm.fast" = TRUE).

Btw, in case you have not seen it: The help page for detect.lindep has an example for within transformation which could fit in your domain (CEOage/tenure simultaneously in FE regression - similar to a pitfall I once ran into as I was not thinking and, hence, created detect.lindep).

tappek commented 2 years ago

plm version 2.4-2 is now CRAN (2021-09-21) containing this fix