create_cov_canonical(singletons = FALSE,hetgrid = NULL) should return a list with two matrices

[pcarbo]

Currently this outputs an empty list, e.g.,

> create_cov_canonical(3,singletons = FALSE,hetgrid = NULL)
named list()

[stephens999]

can we reuse `cov_canonical` from mashr?

…

On Wed, Dec 15, 2021 at 2:33 PM Peter Carbonetto ***@***.***> wrote: Currently this outputs an empty list, e.g., > create_cov_canonical(3,singletons = FALSE,hetgrid = NULL)named list() — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#40>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AANXRRMGX4VYBJTCH7IKPIDURD3TDANCNFSM5KES4JCA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[gaow]

Related to #41 am leaning towards creating / reusing functions to provide the matrices, and repurpose the existing one in mvsusieR as set_mixture_prior to post-process or unify the output from sources outlined in #41, including that from cov_canonical if we reuse it.

@stephens999 we did some post-processing including scaling the cov_canonical by summary statistics if available so we get different values on the diag, and we allow for setting the weights for the matrices in the mixture although the default would be uniform weights.