ycroissant / plm

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

Correct likely bug in pdata.frame #50

Closed MichaelChirico closed 11 months ago

MichaelChirico commented 11 months ago

Detected while stress-testing the new lintr::length_test_linter() (https://github.com/r-lib/lintr/pull/2124).

I lack enough context about this function to offer a regression test, but it should be possible to do so by supplying something with length(index) != 2, then the group.name might populate incorrectly.

tappek commented 11 months ago

great catch, thank you!