irlba
irlba copied to clipboard
irlba's implementation of prcomp loses rotation rownames
Hi, I've rewritten my code to replace prcomp with prcomp_irlba (so much faster 🙌 ), and got an error because, downstream, I call rownames of the rotation matrix, however in irlba's implementation, the rotation matrix has lost the rownames. It would be very useful if you could add back the rownames, to allow more seamless conversion between prcomp and irlba.
Thanks for this suggestion, will put this on the todo list for the next update.
On 4/29/21, leahrosen @.***> wrote:
Hi, I've rewritten my code to replace prcomp with prcomp_irlba (so much faster 🙌 ), and got an error because, downstream, I call rownames of the rotation matrix, however in irlba's implementation, the rotation matrix has lost the rownames. It would be very useful if you could add back the rownames, to allow more seamless conversion between prcomp and irlba.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bwlewis/irlba/issues/59