Eric Larson
Eric Larson
> Likely, we should also change the example structure in the sphinx-gallery repo itself to be a role model. That's quite a bit of code churn, but not really complicated....
It's not just about wheel availability unfortunately. We had related discussions where we ultimately rejected including the pure Python library nibabel in #11564 for different reasons. (As a side note,...
Tracking issue for (implicit) MNE dependencies still lacking binary wheels for `aarch64` (ARM Linux)
Nice catch, there was a spurious failure in the tag build, I restarted and now [2.5.11 is on PyPI](https://pypi.org/project/openmeeg/2.5.11/#files)!
> It would be good to see how the docs look like without that button though. The rendered docs are still available: https://output.circle-artifacts.com/output/job/6b6ff620-ee6c-4671-ab7b-0f559f6dfecc/artifacts/0/html/index.html When the build finishes with a Sphinx...
It looks like you have implemented a new raw class. This is probably going to be difficult to debug, and probably outside the scope of MNE-Python to debug it since...
@lucyleeow I did not follow all discussion so I'll let you review, feel free to merge if you're happy. I can look deeper if it would help to have another...
> and IIUC we don't expect users to pass in an actual class right? just instances? It looks like at least mpl does: https://github.com/matplotlib/matplotlib/blob/9489b938556b16633591c2e829462b1aa4e4a46e/doc/conf.py#L305 https://github.com/matplotlib/matplotlib/blob/9489b938556b16633591c2e829462b1aa4e4a46e/doc/conf.py#L187 https://github.com/matplotlib/matplotlib/blob/9489b938556b16633591c2e829462b1aa4e4a46e/doc/sphinxext/gallery_order.py#L125 Does that change things?
> then MPL's custom sorter class should work too, and the test suite still passes locally. Do you see an easy enough way to modify `tinybuild` to have mpl's use...
... and if we want to be really complete, we should also make the conda-forge recipe for `mne-base` 100% consistent with the minimal reqs in `pyproject.toml` and `mne` at least...
Looks reasonable except this one: ``` times = {"auto": (3, (0.1, 0.8))} ``` Can be more compactly written as ``` times=np.linspace(0.1, 8, 3) ``` So I'd rather just put that...