Bug 16110 - nlme: corCompSymm doesn't call specialized corFactor method
Summary: nlme: corCompSymm doesn't call specialized corFactor method
Status: NEW
Alias: None
Product: R
Classification: Unclassified
Component: Models (show other bugs)
Version: R 3.1.2
Hardware: All All
: P5 minor
Assignee: R-core
URL:
Depends on:
Blocks:
 
Reported: 2014-12-15 07:13 UTC by Kyle Simek
Modified: 2014-12-15 07:13 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle Simek 2014-12-15 07:13:46 UTC 
version: 3.1-118 
library: nlme 
file: R/corStruct.R
line: 1411

Due to a typo in the method name, corCompSymm objects call the base-class's corFactor method instead of its specialized version.  

At line 1411 of R/corStruct.R, the declaration

    corFactor.compSymm <- ...

should be 

    corFactor.corCompSymm <- ...

This bug shouldn't affect correctness, because the base-class method implements the same semantics as the derived class, but I assume the specialized method is likely more efficient.

Disclaimer: I'm new to R and wasn't able to confirm this bug or the proposed fix in a debugger.  I hope maintainers who know more than me can check that my suggested fix makes sense, and apologies in advance if this is off base.