mne-tools / mne-tools/mne-python
ENH: Make _pick_to_idx public
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 3.5k
- Forks
- 1.6k
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 100
Description
@drammock other than
mne.io.constants.FIFF(which is pretty painless to keep) this is the list of what we have to keep around to make sibling packages happy. Do you think we should make some variant of_picks_to_idxpublic?
Originally posted by @larsoner in https://github.com/mne-tools/mne-python/pull/11903#discussion_r1301869325
We'll have to think about the right API and we might not want to expose all options but it should be doable
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 with the discussion in pull request #11903 and review the current _picks_to_idx entry point alongside mne.io.constants.FIFF. Determine which options and behavior should form the public API, and confirm that sibling packages can use it without relying on private details.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100