Eric Larson

Results 114 issues of Eric Larson

In working on streamlining some stuff in #765 I noticed that `ArbitraryContrast` does not seem to be used in any tests: ``` $ git grep "contrasts = " ... mne_bids_pipeline/steps/sensor/_99_group_average.py:...

Just to put on our radar because I hadn't thought about it until today: https://github.com/bids-standard/bep021/issues/8#issuecomment-1604206239 I'm sure @sappelhoff will let us know when we should add something here :)

I thought it would be trivial to get MBP to run FLASH for ds000117, but it wasn't. I ended up needing to read some MRI JSON files and write some...

For example, using a simple config for [ds004024](https://openneuro.org/datasets/ds004024): ``` RuntimeError: Found more than one matching data file for the requested recording. While searching: BIDSPath( root: /home/larsoner/mne_data/ds004024 datatype: eeg basename: sub-CON001_ses-async14ms_task-rest)...

See #645 and in particular https://github.com/openmeeg/openmeeg/actions/runs/8526965136/job/23357331335?pr=645#step:23:4082 Failed test example ``` Test project /home/runner/work/openmeeg/openmeeg/build Start 185: HM-HeadNNa2 1/688 Test #185: HM-HeadNNa2 ...................................Subprocess aborted***Exception: 0.11 sec /home/runner/work/openmeeg/openmeeg/build/apps/om_assemble version 2.5.9999 compiled at Apr...

Eventually maybe we should merge once pytest is fixed since this has a nice `autouse=True` rather than `usefixtures`. But: - [ ] Need to restore removed test file - [...

I think the motivations behind these classes should be documented more clearly. From our doc I couldn't tell: https://qtdocs.pyvista.org/api_reference.html#pyvistaqt.BackgroundPlotter I had to go back to https://github.com/pyvista/pyvistaqt/pull/78 to understand (what I...

We should just strip out all of the Azure `.ci/` -- I think everything is handled on GHA at this point _Originally posted by @banesullivan in https://github.com/pyvista/pyvistaqt/issues/413#issuecomment-1614718073_

See https://github.com/pyvista/pyvistaqt/pull/66#pullrequestreview-1239467934 and related code comments

Supersedes #65 Modernizes some stuff: - Updates for Pytest 8.1 - ~~Use [`packaging`](https://packaging.pypa.io/en/latest/version.html#packaging.version.Version) to get module versions rather than `distutils` (which is gone in 3.12)~~ - Set Python min to...