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

searchlight revisited #253

Draft
wants to merge 35 commits into
base: main
Choose a base branch
from
Draft

searchlight revisited #253

wants to merge 35 commits into from

Conversation

JasperVanDenBosch
Copy link
Contributor

@JasperVanDenBosch JasperVanDenBosch commented Aug 25, 2022

Rethink of the searchlight interface, architecture as well as introducing a surface-based version.

datasets

  • sitek (private)
  • jones-michael-s (private)
  • cichy16 (public)
  • jsong (private)

issues

thoughts and notes

Performance is important for searchlight analysis. The current interface does not readily allow or isn't optimised for an approach where each chunk of data goes through multiple steps in a pipeline; but this would enable parallelisation on an HPC infrastructure. An approach with in-process parallelization using e.g. job lib brings certain complications as well.

@iancharest
Copy link
Contributor

i had to create an intermediate calc_rdm_batch function to deal with the fact that calc_rdm involves recursion and with parallel, this lead to maximum recursive depth errors.

@JasperVanDenBosch JasperVanDenBosch changed the title surface searchlight searchlight revisited Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants