alyakin314
alyakin314
@bdpedigo, in short: we (as in you and I) had no idea of whether it leads to a valid / consistent test in the graph setting, and whether is it...
Both seem options appropriate to me; removing the intermediate categorizations seem more user-friendly than nesting them, but both have pros and cons obviously.
i also wanted to add in that it might be time for us to re-arrange references in some sort of a logical order
im happy to review this as well if this ever happens
> This is our forum for asking whatever network question you'd like! No need to feel shy - we're happy to talk about graphs! > > I have been trying...
> When I set n_components to None so it would use the Zhu and Ghodsie selection procedure I got this error: > > "n_components must be
> I am calculating the latent distribution test for the cross-product of the following graphs: > > GraphA has 1695391 edges > GraphB has 463153 edges > GraphC has 2101995...
> Line lengths are short (recommended
> For the PR itself portion, maybe mention that new features should include proof of effectiveness, like a jupyter notebook that replicates a figure from a paper already here: https://github.com/microsoft/graspologic/blob/dev/CONTRIBUTING.md...
> @dwaynepryce @j1c @asaadeldin11 @alyakin314 any thoughts on the above? I'd like to finalize this (at least for the purposes of NDD) by the end of the week so LMK....