matrixStats: Upcoming version requires using useNames = TRUE in two places #657
Comments
Closed
4 tasks
|
Henrik, this is great.
Thanks for your help on suggestions for making GenEst work with the new matrixStats.
Your work goes above and beyond the call of duty.
Much appreciated.
…-Dan
------- Original Message -------
On Monday, November 14th, 2022 at 7:06 PM, Henrik Bengtsson ***@***.***> wrote:
Hello. The matrixStats package will soon be updated to output row and column names consistently. The current behavior is useNames = NA for backward compatibility, but we'll switch to useNames = FALSE soon, cf. [HenrikBengtsson/matrixStats#226](HenrikBengtsson/matrixStats#226).
The switch to useNames = FALSE will break GenEst. I've created PR [#656](#656) that fixes this. All that was needed is to specify useNames = TRUE in summary.splitFull() in two places.
PS. I noticed that you submitted GenEst 1.4.7 just two hours ago. What's the chance for that? Oh, well. If you happen to get a pushback on that submission, please consider incorporating this PR in your resubmission.
—
Reply to this email directly, [view it on GitHub](#657), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AHFSXPZXVTJWUW5FMAFPLS3WIL44DANCNFSM6AAAAAASAO3NEM).
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
ddalthorp
pushed a commit
that referenced
this issue
Nov 22, 2022
|
I see a commit on this. Do you have an ETA for submitting this to CRAN? |
|
23 Nov 2022
…------- Original Message -------
On Wednesday, January 11th, 2023 at 7:10 AM, Henrik Bengtsson ***@***.***> wrote:
I see a commit on this. Do you have an ETA for submitting this to CRAN?
—
Reply to this email directly, [view it on GitHub](#657 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AHFSXP6WJHDVLK775S4ETALWR3EQDANCNFSM6AAAAAASAO3NEM).
You are receiving this because you commented.Message ID: ***@***.***>
|
|
Oh, so you're saying it's already on CRAN. I missed that because this issue was still open. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello. The matrixStats package will soon be updated to output row and column names consistently. The current behavior is
useNames = NAfor backward compatibility, but we'll switch touseNames = FALSEsoon, cf. HenrikBengtsson/matrixStats#226.The switch to
useNames = FALSEwill break GenEst. I've created PR #656 that fixes this. All that was needed is to specifyuseNames = TRUEinsummary.splitFull()in two places.PS. I noticed that you submitted GenEst 1.4.7 to CRAN just two hours ago. What's the chance for that? Oh, well. If you happen to get a pushback on that submission, please consider incorporating this PR in your resubmission.
The text was updated successfully, but these errors were encountered: