Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update matrix and vector initialization #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eddelbuettel
Copy link
Contributor

Very sorry about apparently having forgotten one change set in pull request #3 I sent a few months ago, and which you kindly merged, and updated at CRAN. Because of my oversight, we will need another round. It would be much appreciated if you could do another (trivial, my bad for the extra work) round at CRAN.

(We could upate the initialization of G just above similarly. C++11 brace initialization work with x = { 1, 2, 3 }; where vector dimensions are automatic, for matrices it works row-wise as I explicitly tested so M = { {1, 2}, {3, 4} }; becomes a matrix with rows {1, 2} and {3, 4}.

Copy link
Owner

@bpfaff bpfaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Dirk, a happy new year and many thanks for the patch. Best wishes, Bernhard

@eddelbuettel
Copy link
Contributor Author

Thanks, and Happy New Year to you too! Thanks for the quick approval, merge and upload by April would be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants