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

External regressors with NaNs will crash decision tree #1152

Open
tsalo opened this issue Nov 22, 2024 · 0 comments
Open

External regressors with NaNs will crash decision tree #1152

tsalo opened this issue Nov 22, 2024 · 0 comments
Labels
bug issues describing a bug or error found in the project

Comments

@tsalo
Copy link
Member

tsalo commented Nov 22, 2024

Summary

I tried building a decision tree to use motion parameters from an fMRIPrep confounds file. The problem is that relative motion measures (e.g., framewise displacement) have a NaN in the first row of the confounds because they're derivatives. We could leave it up to users to deal with this, but I think an easy workaround on our end would be to backfill NaNs automatically.

Additional Detail

We could also raise a warning if NaNs are present to ensure users know what's happening.

@tsalo tsalo added the bug issues describing a bug or error found in the project label Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issues describing a bug or error found in the project
Projects
None yet
Development

No branches or pull requests

1 participant