91473147e57464173dedb00ce20f3d5aefd883bd,mne/preprocessing/tests/test_ica.py,,test_ica_noop,#Any#Any#Any#,168

Before Change

    ica.fit(raw)
    raw_new = ica.apply(raw.copy())
    // not a no-op
    if max_pca_components == 8:  // XXX this fails: or n_pca_components == 8:
        assert not np.allclose(raw.get_data(), raw_new.get_data())
        return
    // XXX this case should warn?
    if max_pca_components == 10:
        assert ica.max_pca_components_ == 10
        // XXX setting n_components to something safe prevents the blowup, maybe
        // we don"t need max_pca_components as float? For example setting the

After Change

        ctx = pytest.warns(RuntimeWarning, match=".*unstable.*integer <= 9")
        bad = True  // pinv will fail
    elif n_components == 0.9999 and n_pca_components == 8:
        ctx = pytest.raises(RuntimeError, match="requires 9 PCA values.*but")
        bad = "exit"
    else:
        bad = False  // pinv will not fail
        ctx = nullcontext()
    with ctx:
        ica.fit(raw)
    assert ica._max_pca_components is None
    if bad == "exit":
        return
    raw_new = ica.apply(raw.copy())

Instances