Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow pandas.Series as valid input for second_level_input in SecondLevelModel class #2411

Closed
JohannesWiesner opened this issue Feb 22, 2020 · 1 comment · Fixed by #4070
Closed
Labels
Enhancement for feature requests GLM Issues/PRs related to the nilearn.glm module.

Comments

@JohannesWiesner
Copy link
Contributor

It would be nice, if a pandas.Series would also be allowed as valid input for second_level_input in SecondLevelModel. Currently, I have to convert my series containing the NIFTI filepaths to a list. However, I guess most of the time you work with a pandas.DataFrame containing information about your subjects (ID, age, filepaths, etc.) and you would like to provide second_level_input directly with the column containing the filepaths (e.g. SecondLevelModel.fit(second_level_input=my_dataframe['filepaths')

@bthirion
Copy link
Member

Agreed !
For that one, I think we're going to implement in nilearn.stats (note that nistats will be frozen soonish, see e.g. #2299).
Best,

@kchawla-pi kchawla-pi transferred this issue from nilearn/nistats Apr 2, 2020
@kchawla-pi kchawla-pi transferred this issue from nilearn/nilearn Apr 2, 2020
@kchawla-pi kchawla-pi transferred this issue from nilearn/nistats Apr 2, 2020
@kchawla-pi kchawla-pi added the GLM Issues/PRs related to the nilearn.glm module. label Apr 2, 2020
@Remi-Gau Remi-Gau added this to the release 0.11.0 milestone Oct 7, 2023
@Remi-Gau Remi-Gau added the Enhancement for feature requests label Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement for feature requests GLM Issues/PRs related to the nilearn.glm module.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants