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

Fix two seeds example #5

Merged
merged 2 commits into from
Jul 18, 2017
Merged

Fix two seeds example #5

merged 2 commits into from
Jul 18, 2017

Conversation

helske
Copy link
Contributor

@helske helske commented Jul 16, 2017

The example used a function from another demo, the matrix of results had one extra column, and no seed was used for second engine.

Use second seed for eng2 and return n x 2 matrix instead of n x 3.
@coatless
Copy link
Collaborator

Thanks again! I'll also update the code in src/.

@coatless coatless merged commit 0f173e8 into coatless-rpkg:master Jul 18, 2017
@helske
Copy link
Contributor Author

helske commented Jul 18, 2017

No problem, glad to help out. I forgot the examples were also part of cpp files.

@coatless coatless added the bug label Jul 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants