Adam Li

Results 474 comments of Adam Li
trafficstars

Hey @aminmkhan great feature! I just had an issue that the `et al.` was not showing up when I print the bibliography. I was wondering if this is somehow getting...

That's awesome! Per our discussion in https://github.com/mne-tools/mne-connectivity/issues/57 do we still want the basic spectral measures inside mne-connectivity with a lightweight wrapper to Frites? If so, would it be okay with...

Suite of tests we should migrate over from dodiscover are: - [ ] Chi-square - [ ] G^2 (note there might be discrepancies between G^2 implementation vs Chi-square general approach)...

A related issue is including a few very short-examples of the API in the README. cc: @bloebp in case you have time to add w/e you were thinking in

Can someone add the CircleCI token to the pywhy org? By following the steps here? This would enable circleCI artifact redirect on all repos that use it: dodiscover, pywhy-graphs, pywhy-stats...

> hi @dschult, `compute_v_structures` currently returns colliders, not v-structures. Renaming it to `compute_colliders` and adapting a little the docstring would already be sufficient as far as I can tell (could...

I confirm I am seeing the same issue: https://github.com/neurodata/scikit-tree/actions/runs/8895864721/job/24427188361?pr=264 Is there any temporary fix? Does downgrading to v4 work?

> ### `mne_icalabel` > As described in the readme, this package is not only a port of the popular `ICLabel` network for EEG but could host other models for EEG,...

So I think in line with something like sklearn, we have the following semantics: - `fit()`: fits parameters to a specific dataset. The ICLabel model is "fixed", so the only...

Ah I see, so the proposal here is to migrate this altogether into MNE, so that way it's structured within the `info` object or something. I'm in favor of then...