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

Vignette notation discrepancy #19

Open
yunqiyang0215 opened this issue Jun 30, 2023 · 6 comments
Open

Vignette notation discrepancy #19

yunqiyang0215 opened this issue Jun 30, 2023 · 6 comments

Comments

@yunqiyang0215
Copy link
Collaborator

ud_init() takes dat as input, and dat can either from mash_set_data() or X. But X is not mentioned before using it. Same problem occurs in

U_scaled = list(indep = diag(ncol(X)), equal = matrix(1, ncol(X), ncol(X)) + 0.0001 *
    diag(ncol(X)))

See here: https://stephenslab.github.io/udr/reference/ud_init.html

pcarbo added a commit that referenced this issue Jun 30, 2023
@pcarbo
Copy link
Member

pcarbo commented Jun 30, 2023

@yunqiyang0215 Can you please test the new version of ud_init with a data matrix and a mash data object to make sure it works with both those data types?

@yunqiyang0215
Copy link
Collaborator Author

Hi @pcarbo , I just tested the implementation and it works. But I think the documentation about the function ud_init() also needs to be updated.

@pcarbo
Copy link
Member

pcarbo commented Jun 30, 2023

Better now?

@yunqiyang0215
Copy link
Collaborator Author

Yes. A minor point, for the explanation of dat.

An n x m data matrix, in which each row of the matrix is an m-dimensional data point, or a “mash” object, for example created by [mash_set_data](https://rdrr.io/pkg/mashr/man/mash_set_data.html). When X is a matrix it should have at least 2 rows and at least 2 columns.

We should also replace X with dat?

@pcarbo
Copy link
Member

pcarbo commented Jun 30, 2023

Yes, good catch. This should now be fixed. I didn't update the pkgdown webpage with the change, but I'm sure we will have another opportunity to do so soon.

@yunqiyang0215
Copy link
Collaborator Author

I see. Sounds good. Thanks, Peter!

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

No branches or pull requests

2 participants