Jérôme Dockès
Jérôme Dockès
> @jeromedockes > want me to take over this one? Hi Remi sorry for the delay. Please feel free to do so, no problem but actually I think it might...
> @Remi-Gau we need to note that https://github.com/nilearn/nilearn/pull/4120 needs to be reverted if we release without the surface API. maybe we could avoid reverting it. to avoid importing the experimental...
Yasmin ***@***.***> writes: >> As far as I can tell the SurfaceMasker does not include a `generate_report` method like the other maskers. > > I define it and raise NotImplemetedError...
> would literal have to type "experimental". indeed that is more likely to catch their attention than any warning
> ... and for the time being, we represent meshes through dictionaries. or rather collections of meshes containing one or more hemispheres. each individual mesh is still represented by a...
> I would say mostly as signal to users (and future us) that this is coming. ok my 2 cents is stating it exists by defining it and then surprise...
why not use `image.load_img(img).get_fdata()` ? or `nibabel.load_img(img).get_fdata()` ?
If I understand correctly your PR will add a switch inside `nilearn._utils.niimg._get_data`?
Thanks! could you provide a small example showing what is the problem? Passing a list of images to `fit_transform` seems to work for me, as well as passing the atlas...
weak -1 for me. even though it's been around for a few releases I don't see the walrus operator being used that often; I wouldn't be surprised if some contributors...