Lazy loading in `BaseBIDSDataset`
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.1k
- Forks
- 264
- Avg merge
- 1d 13m
- Merged PRs (30d)
- 23
Description
# Data needs to be preloaded for the filtering step of paradigms
raw.load_data()
Okay for now, but we need to remove this step. Full lazy in the future, or only load at the filtering stage.
Originally posted by @bruAristimunha in https://github.com/NeuroTechX/moabb/pull/724#discussion_r1989525921
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the BaseBIDSDataset entry point and trace the filtering path around raw.load_data(). The issue does not name a file or test and leaves two possible outcomes—full lazy loading or loading only during filtering—so that scope needs clarification first. Done means removing the current eager preload without breaking paradigm filtering.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100